Skip to content

[bug Linux?] Make does not honor comment symbol in addon_config.mk #620

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

Open
artificiel opened this issue Mar 9, 2025 · 3 comments
Open

Comments

@artificiel
Copy link
Contributor

trying to disable a line such as

#       ADDON_SOURCES = DeckLinkAPIDispatch.cpp

does not work, it is still considered by the build system. this has the potential to break many addons.

@dimitre
Copy link
Member

dimitre commented Mar 10, 2025

I'm wondering if this is a Makefile issue instead of PG
If yes, maybe it is happening for a while now.
it would be great to test on old releases to see if it is like this.

@artificiel
Copy link
Contributor Author

ah! yes: using make, only editing the addon_config without re-processing with PG can toggle the error. it seems to go back to 0.11.2 at least.

for a moment i was confused as the comments seemed to work again, but i've narrowed it to the tabs... so

these are working:

#Something
# Something
 # Something

but as soon as a tab is introduced, it stops being commented out

#<tab>This is not commented out
# <tab>nor is this
# <tab> or this
#or even<tab>this // this one actually complains about missing separator

i'm surprised this was not causing problems before... but maybe something else is at play?

in any case it's indeed not PG; does github enable transferring issues between repos from PG to OF?

@dimitre dimitre changed the title [bug Linux?] PG does not honor comment symbol in addon_config.mk [bug Linux?] Make does not honor comment symbol in addon_config.mk Mar 10, 2025
@dimitre
Copy link
Member

dimitre commented Mar 10, 2025

I suppose it is like this forever and you are the first to find.

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

No branches or pull requests

2 participants