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

Wrap Makefile script's directory variables in quotes #808

Merged
merged 1 commit into from
Dec 13, 2022

Conversation

sneurlax
Copy link
Contributor

This pull request wraps the directory variables on line 619 of Makefile.in in quotes so as to avoid make install failure due to a script parsing error as seen in #807

@sneurlax
Copy link
Contributor Author

These changes have not been tested so as to not affect other platforms/toolchains; a reviewer ought to make sure that the main hosts/targets aren't compromised.

sneurlax added a commit to sneurlax/flutter_libwownero that referenced this pull request Dec 13, 2022
Copy link
Member

@wcawijngaards wcawijngaards left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change looks good to me. It should be fine for building on other targets.

@wcawijngaards wcawijngaards merged commit 5204c14 into NLnetLabs:master Dec 13, 2022
wcawijngaards added a commit that referenced this pull request Dec 13, 2022
- Merge #808: Wrap Makefile script's directory variables in quotes.
@wcawijngaards
Copy link
Member

Thanks for the fix! Merged it into the code repository.

jedisct1 added a commit to jedisct1/unbound that referenced this pull request Dec 13, 2022
* nlnet/master:
  - Updates for NLnetLabs#461 (Add max-query-restarts option).
  - Expose 'max-sent-count' as a configuration option; the   default value retains Unbound's behavior.
  - Expose 'statistics-inhibit-zero' as a configuration option; the   default value retains Unbound's behavior.
  - Fix to wrap Makefile scripts directory in quotes for uninstall.
  Changelog note for NLnetLabs#808 - Merge NLnetLabs#808: Wrap Makefile script's directory variables in quotes.
  wrap directory variables in quotes
  Fix date.
  - Fix NLnetLabs#773: When used with systemd-networkd, unbound does not start   until systemd-networkd-wait-online.service times out.
  - Clear documentation for interactivity between the subnet module and   the serve-expired and prefetch configuration options.
  - Add SVCB and HTTPS to the types removed by 'unbound-control flush'.
  - Fix NLnetLabs#782: Segmentation fault in stats.c:404.
  Changelog entry for NLnetLabs#720
  Document max-query-restarts option
  Use max-query-restarts in iterative resolver
  Add max-query-restarts to grammar and lexer
  Add max-query-restarts config parameter
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