Skip to content

Makefile.pre.in: commoninstall: reformat #75371

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
dilyanpalauzov mannequin opened this issue Aug 12, 2017 · 1 comment
Closed

Makefile.pre.in: commoninstall: reformat #75371

dilyanpalauzov mannequin opened this issue Aug 12, 2017 · 1 comment
Labels
3.9 only security fixes 3.10 only security fixes 3.11 only security fixes build The build process and cross-build easy type-bug An unexpected behavior, bug, or error

Comments

@dilyanpalauzov
Copy link
Mannequin

dilyanpalauzov mannequin commented Aug 12, 2017

BPO 31188

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = None
closed_at = None
created_at = <Date 2017-08-12.17:58:52.659>
labels = ['easy', 'type-bug', '3.9', '3.10', '3.11', 'build']
title = 'Makefile.pre.in: commoninstall: reformat'
updated_at = <Date 2022-01-04.22:33:51.731>
user = 'https://bugs.python.org/dilyanpalauzov'

bugs.python.org fields:

activity = <Date 2022-01-04.22:33:51.731>
actor = 'iritkatriel'
assignee = 'none'
closed = False
closed_date = None
closer = None
components = ['Build']
creation = <Date 2017-08-12.17:58:52.659>
creator = 'dilyan.palauzov'
dependencies = []
files = []
hgrepos = []
issue_num = 31188
keywords = ['easy']
message_count = 1.0
messages = ['300204']
nosy_count = 1.0
nosy_names = ['dilyan.palauzov']
pr_nums = []
priority = 'normal'
resolution = None
stage = None
status = 'open'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue31188'
versions = ['Python 3.9', 'Python 3.10', 'Python 3.11']

Linked PRs

@dilyanpalauzov
Copy link
Mannequin Author

dilyanpalauzov mannequin commented Aug 12, 2017

When @FRAMEWORKALTINSTALLLAST@ expands to "", the last line of recipe commoninstall in the resulting Makefile consists solely of two tabs, for which emacs complains as "suspicious line".

diff --git a/Makefile.pre.in b/Makefile.pre.in
--- a/Makefile.pre.in
+++ b/Makefile.pre.in
@@ -1069,8 +1069,7 @@ altinstall: commoninstall
 
 commoninstall:  @FRAMEWORKALTINSTALLFIRST@ \
                altbininstall libinstall inclinstall libainstall \
-               sharedinstall oldsharedinstall altmaninstall \
-               @FRAMEWORKALTINSTALLLAST@
+               sharedinstall oldsharedinstall altmaninstall @FRAMEWORKALTINSTALLLAST@
 
 # Install shared libraries enabled by Setup
 DESTDIRS=      $(exec_prefix) $(LIBDIR) $(BINLIBDEST) $(DESTSHARED)

@dilyanpalauzov dilyanpalauzov mannequin added 3.7 (EOL) end of life build The build process and cross-build labels Aug 12, 2017
@iritkatriel iritkatriel added easy 3.9 only security fixes 3.10 only security fixes 3.11 only security fixes type-bug An unexpected behavior, bug, or error and removed 3.7 (EOL) end of life labels Jan 4, 2022
@ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
ambv pushed a commit that referenced this issue Jul 22, 2023
…ALTINSTALLLAST (#107035)

in the case of an empty FRAMEWORKALTINSTALLLAST, this patch prevents leaving
an astray linebreak and two tabs in the resulting Makefile.

Before change:
```
.PHONY: commoninstall
commoninstall:  check-clean-src  \
		altbininstall libinstall inclinstall libainstall \
		sharedinstall altmaninstall \

```

After change (with empty FRAMEWORKALTINSTALLLAST):
```
.PHONY: commoninstall
commoninstall:  check-clean-src  \
		altbininstall libinstall inclinstall libainstall \
		sharedinstall altmaninstall
```
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jul 22, 2023
…MEWORKALTINSTALLLAST (pythonGH-107035)

in the case of an empty FRAMEWORKALTINSTALLLAST, this patch prevents leaving
an astray linebreak and two tabs in the resulting Makefile.

Before change:
```
.PHONY: commoninstall
commoninstall:  check-clean-src  \
		altbininstall libinstall inclinstall libainstall \
		sharedinstall altmaninstall \

```

After change (with empty FRAMEWORKALTINSTALLLAST):
```
.PHONY: commoninstall
commoninstall:  check-clean-src  \
		altbininstall libinstall inclinstall libainstall \
		sharedinstall altmaninstall
```
(cherry picked from commit 9c38206)

Co-authored-by: Moritz Neeb <[email protected]>
ambv pushed a commit to ambv/cpython that referenced this issue Jul 22, 2023
…pty FRAMEWORKALTINSTALLLAST (pythonGH-107035)

in the case of an empty FRAMEWORKALTINSTALLLAST, this patch prevents leaving
an astray linebreak and two tabs in the resulting Makefile.

Before change:
```
.PHONY: commoninstall
commoninstall:  check-clean-src  \
		altbininstall libinstall inclinstall libainstall \
		sharedinstall altmaninstall \

```

After change (with empty FRAMEWORKALTINSTALLLAST):
```
.PHONY: commoninstall
commoninstall:  check-clean-src  \
		altbininstall libinstall inclinstall libainstall \
		sharedinstall altmaninstall
```
(cherry picked from commit 9c38206)

Co-authored-by: Moritz Neeb <[email protected]>
ambv added a commit that referenced this issue Jul 23, 2023
…AMEWORKALTINSTALLLAST (GH-107035) (#107050)

in the case of an empty FRAMEWORKALTINSTALLLAST, this patch prevents leaving
an astray linebreak and two tabs in the resulting Makefile.

Before change:
```
.PHONY: commoninstall
commoninstall:  check-clean-src  \
		altbininstall libinstall inclinstall libainstall \
		sharedinstall altmaninstall \

```

After change (with empty FRAMEWORKALTINSTALLLAST):
```
.PHONY: commoninstall
commoninstall:  check-clean-src  \
		altbininstall libinstall inclinstall libainstall \
		sharedinstall altmaninstall
```
(cherry picked from commit 9c38206)

Co-authored-by: Moritz Neeb <[email protected]>
ambv pushed a commit that referenced this issue Jul 23, 2023
…AMEWORKALTINSTALLLAST (GH-107035) (#107049)

gh-75371: reformat Makefile.pre.in to accommodate for empty FRAMEWORKALTINSTALLLAST (GH-107035)

in the case of an empty FRAMEWORKALTINSTALLLAST, this patch prevents leaving
an astray linebreak and two tabs in the resulting Makefile.

Before change:
```
.PHONY: commoninstall
commoninstall:  check-clean-src  \
		altbininstall libinstall inclinstall libainstall \
		sharedinstall altmaninstall \

```

After change (with empty FRAMEWORKALTINSTALLLAST):
```
.PHONY: commoninstall
commoninstall:  check-clean-src  \
		altbininstall libinstall inclinstall libainstall \
		sharedinstall altmaninstall
```
(cherry picked from commit 9c38206)

Co-authored-by: Moritz Neeb <[email protected]>
@hugovk hugovk closed this as completed Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.9 only security fixes 3.10 only security fixes 3.11 only security fixes build The build process and cross-build easy type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

No branches or pull requests

2 participants