You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Use via Inline::C with compiler flags is broken #430
An Alien that sets cflags to something like -Ifoo -Dbar breaks when used via use Inline with.
The reason is that Alien::Base’s Inline() method puts all these flags into INC, while Inline::C quotes spaces in INC when flags other than -I appear inside it. The INC value that Inline::C passes on to EU:MM becomes "-Ifoo -Dbar". Consequently the compiler doesn’t look for a directory named foo, it looks for a directory named foo -Dbar instead.
A somewhat hacky workaround might be to include at least one double quote in cflags, which would currently prevent Inline::C from adding further quotes in its quote_spaces() function.
But I think the proper solution for Alien::Base would rather be to revert #109, putting the -D and other flags into CCFLAGSEX instead.
Yeah my preference would be to revert #109. However I think we may need to try to fix ExtUtils::Depends or maybe just return the broken config when ExtUtils::Depends calls the AB Inline method.
I've deployed #443 as a dev version 2.89_01. Can you give it a try? Either way will probably do a prod release with this change in the next couple weeks or so.
Activity
johannessen commentedon Oct 24, 2025
plicease commentedon Sep 12, 2026
Yeah my preference would be to revert #109. However I think we may need to try to fix
ExtUtils::Dependsor maybe just return the broken config whenExtUtils::Dependscalls the ABInlinemethod.plicease commentedon Sep 21, 2026
I've deployed #443 as a dev version
2.89_01. Can you give it a try? Either way will probably do a prod release with this change in the next couple weeks or so.johannessen commentedon Sep 23, 2026
Frankly I am not interested in “giving LLM output a try.”
plicease commentedon Sep 23, 2026
okay then.