Skip to content

Makefile: comment on seemingly unused variable#534

Closed
vbatts wants to merge 1 commit intoopencontainers:masterfrom
vbatts:unused_makefile_var
Closed

Makefile: comment on seemingly unused variable#534
vbatts wants to merge 1 commit intoopencontainers:masterfrom
vbatts:unused_makefile_var

Conversation

@vbatts
Copy link
Copy Markdown
Member

@vbatts vbatts commented Aug 19, 2016

Signed-off-by: Vincent Batts vbatts@hashbangbash.com

@wking
Copy link
Copy Markdown
Contributor

wking commented Aug 19, 2016

SHELL was added in f3fdf03 (Makefile: prefer bash, 2016-05-25, #455)
referencing #454. I am in favor of a Makefile that only needs a POSIX
shell, but I'd like to understand any POSIX-shell-incompatibilities.
I would have preferred to have that information before we added the
variable 1, but I guess I'm ok removing it, waiting for someone to
reopen #454, and then collecting that information before re-adding
SHELL (or ideally, fixing the POSIX-shell-incompatible command).

@vbatts
Copy link
Copy Markdown
Member Author

vbatts commented Aug 19, 2016

Oh oh oh
This commit should instead be.... A comment

On Fri, Aug 19, 2016, 12:25 W. Trevor King notifications@github.com wrote:

SHELL was added in f3fdf03 (Makefile: prefer bash, 2016-05-25, #455)
referencing #454. I am in favor of a Makefile that only needs a POSIX
shell, but I'd like to understand any POSIX-shell-incompatibilities.
I would have preferred to have that information before we added the
variable 1, but I guess I'm ok removing it, waiting for someone to
reopen #454, and then collecting that information before re-adding
SHELL (or ideally, fixing the POSIX-shell-incompatible command).


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#534 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEF6fuOapqs7JJfkqy0sbThY2U8MYvQks5qhdkBgaJpZM4JolHR
.

Signed-off-by: Vincent Batts <vbatts@hashbangbash.com>
@vbatts vbatts force-pushed the unused_makefile_var branch from f2ce877 to e044834 Compare August 26, 2016 14:40
@vbatts vbatts changed the title Makefile: remove unused variable Makefile: comment on seemingly unused variable Aug 26, 2016
@vbatts
Copy link
Copy Markdown
Member Author

vbatts commented Aug 26, 2016

updated PTAL

@wking
Copy link
Copy Markdown
Contributor

wking commented Aug 26, 2016 via email

@vbatts vbatts closed this Aug 29, 2016
@vbatts vbatts deleted the unused_makefile_var branch August 29, 2016 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants