Skip to content

Conversation

SoapGentoo
Copy link
Contributor

Hi @raboof
I'd like to upstream the patches for notion we need in Gentoo.

* `LDFLAGS` needs to come very early on the command line,
  since most linker flags only affect object files after it,
  not before.
`$(CC)` will default to `cc` when not specified.
* These are specified in POSIX and actually lack
  `CPPFLAGS` when using GNU Make.
Copy link
Owner

@raboof raboof left a comment

Choose a reason for hiding this comment

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

changes look good, verified the build also still works for the Nix package, thanks!

@raboof raboof merged commit 0a68b33 into raboof:master Jun 12, 2022
@SoapGentoo SoapGentoo deleted the gentoo-patches branch June 12, 2022 20:05
@@ -65,24 +67,24 @@ LUAC=$(LUA_DIR)/bin/luac

ifndef LUA_MANUAL

Copy link
Collaborator

Choose a reason for hiding this comment

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

Makes me wonder if this entire section isn't actually duplicated here from build/lua-detect.mk, libextl lives in-tree.

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.

3 participants