Skip to content
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

all: sync and dedup the creation of the SYSNAME and OBJTYPE env vars #321

Merged

Conversation

nsajko
Copy link
Contributor

@nsajko nsajko commented Jan 4, 2020

This should prevent the issues of dist/buildmk and src/mkhdr getting out
of synchronization yet again.

I also add a rule for arm64 to the OBJTYPE sed command.

Fixes #243
Fixes #320

Change-Id: I60f69a1f32b5ed5ae5ac8a1659c38e29debed005

@rastagraffix
Copy link

I can't verify 'til Monday when I get to the office. But it's a trivial fix so I wouldn't worry ...

This should prevent the issues of dist/buildmk and src/mkhdr getting out
of synchronization yet again.

I also add a rule for arm64 to the OBJTYPE sed command.

Fixes 9fans#243
Fixes 9fans#320

Change-Id: I60f69a1f32b5ed5ae5ac8a1659c38e29debed005
@nsajko nsajko force-pushed the SYSNAME_OBJTYPE_merge_and_deduplication branch from 98095fc to 582ab1c Compare January 5, 2020 22:27
@dancrossnyc dancrossnyc merged commit 6f5bd96 into 9fans:master Jan 7, 2020
@rsc
Copy link
Contributor

rsc commented Jan 10, 2020

Please use shorter file names. When I saw a file named buildEnvironmentVariables in my tree I thought some system tool had left junk behind somehow.

rsc added a commit that referenced this pull request Jan 10, 2020
Also note that the file must be valid shell and mk input.
For #321.
@rsc
Copy link
Contributor

rsc commented Jan 10, 2020

I pushed a CL moving the file to src/mkenv.

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.

fix: build fails on freebsd arm64 Support aarch64
4 participants