-
Notifications
You must be signed in to change notification settings - Fork 1k
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
make linker script generator more portable #732
Comments
karlp
added a commit
that referenced
this issue
Nov 19, 2017
Until #732 has been fixed, it's not enough to just have it in the README that you need GNU awk. Explicitly use the "gawk" command name. This exists on (sane) systems that have gawk as awk, and for systems that use mawk as default, the gawk name should also exist. This should make it significantly easier to diagnost the cause of build problems.
noonien
added a commit
to noonien/libopencm3
that referenced
this issue
Nov 21, 2017
This removes the dependency on python and should fix libopencm3#732
noonien
added a commit
to noonien/libopencm3
that referenced
this issue
Nov 21, 2017
This removes the dependency on python and should fix libopencm3#732
gregschlom
pushed a commit
to Keenwawa/libopencm3
that referenced
this issue
Dec 4, 2017
Until libopencm3#732 has been fixed, it's not enough to just have it in the README that you need GNU awk. Explicitly use the "gawk" command name. This exists on (sane) systems that have gawk as awk, and for systems that use mawk as default, the gawk name should also exist. This should make it significantly easier to diagnost the cause of build problems.
karlp
pushed a commit
to karlp/libopencm3
that referenced
this issue
Mar 2, 2018
This removes the dependency on python and should fix libopencm3#732
karlp
pushed a commit
to karlp/libopencm3
that referenced
this issue
Mar 2, 2018
This removes the dependency on python and should fix libopencm3#732
karlp
pushed a commit
to karlp/libopencm3
that referenced
this issue
Mar 2, 2018
This removes the dependency on awk and should fix libopencm3#732 python was already a dependency, for the irq generation
karlp
pushed a commit
to karlp/libopencm3
that referenced
this issue
Mar 4, 2018
This removes the dependency on awk and should fix libopencm3#732 python was already a dependency, for the irq generation
karlp
pushed a commit
to karlp/libopencm3
that referenced
this issue
Mar 14, 2018
This removes the dependency on awk and should fix libopencm3#732 python was already a dependency, for the irq generation
karlp
pushed a commit
to karlp/libopencm3
that referenced
this issue
Apr 4, 2018
This removes the dependency on awk and should fix libopencm3#732 python was already a dependency, for the irq generation
abhisit
pushed a commit
to abhisit/libopencm3
that referenced
this issue
Aug 15, 2018
This removes the dependency on awk and should fix libopencm3#732 python was already a dependency, for the irq generation
BOJIT
pushed a commit
to BOJIT/PlatformIO-libopencm3
that referenced
this issue
Jan 30, 2021
Interim fix. See libopencm3#732 and libopencm3#731
BOJIT
pushed a commit
to BOJIT/PlatformIO-libopencm3
that referenced
this issue
Jan 30, 2021
Until libopencm3#732 has been fixed, it's not enough to just have it in the README that you need GNU awk. Explicitly use the "gawk" command name. This exists on (sane) systems that have gawk as awk, and for systems that use mawk as default, the gawk name should also exist. This should make it significantly easier to diagnost the cause of build problems.
BOJIT
pushed a commit
to BOJIT/PlatformIO-libopencm3
that referenced
this issue
Jan 30, 2021
This removes the dependency on awk and should fix libopencm3#732 python was already a dependency, for the irq generation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
If it's going to stay awk, it should at least be portable awk and not require GNU awk. See #731 and probably #703
Alternatively, replace it with a python script, as python is already a dependency, and less likely to have insane portability and legibility issues.
The text was updated successfully, but these errors were encountered: