Skip to content

(add) support for automake conditional 'if' directive#90

Open
TApplencourt wants to merge 2 commits into
EbodShojaei:mainfrom
TApplencourt:add-automake-if-conditional
Open

(add) support for automake conditional 'if' directive#90
TApplencourt wants to merge 2 commits into
EbodShojaei:mainfrom
TApplencourt:add-automake-if-conditional

Conversation

@TApplencourt
Copy link
Copy Markdown
Contributor

@TApplencourt TApplencourt commented Apr 2, 2026

We use automake, that me we have makefile.am. Automake support if in their .am:

https://www.gnu.org/software/automake/manual/html_node/Usage-of-Conditionals.html

so I made a little change so that if statement are not indented, so we can use mbake : ) And have a nice linted makefile!

Thanks again!

@EbodShojaei EbodShojaei self-assigned this Apr 10, 2026
@EbodShojaei
Copy link
Copy Markdown
Owner

Thank you! This will be reviewed for inclusion in next version release.

@EbodShojaei
Copy link
Copy Markdown
Owner

@TApplencourt Sorry about this merge conflict due to the recent merge to main. Feel free to push a new commit; I’ll update and resolve the merge locally on my working branch once this is reviewed.

@TApplencourt TApplencourt force-pushed the add-automake-if-conditional branch from 7c03e48 to 76d2d58 Compare April 15, 2026 07:11
@TApplencourt
Copy link
Copy Markdown
Contributor Author

No problem! Resolved the merge conflict. Should be good now.

As usual thanks for merging all those automake thingy, and for the tool :)

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