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

MultiTerm: Minor updates of the README file #204

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

codebrainz
Copy link
Member

No description provided.

For detailed instructions, see the INSTALL file of the Geany-Plugins package
this plugin is part of.
If you want to hide these warnings, add the appropriate `-Wno-XXX` compiler
flags to the ``MULTITERM_CFLAGS`` environment variable when configuring the
Copy link
Member

Choose a reason for hiding this comment

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

won't that conflict with the pkg-config values? if MULTITERM_CFLAGS is a pkg-config thing I believe setting it will override the pkg-config value.

Copy link
Member Author

Choose a reason for hiding this comment

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

I dunno, I just looked at configure --help, would it be better to say VALAFLAGS or something else, and using -Xfoo to pass foo to CC?

Copy link
Member Author

Choose a reason for hiding this comment

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

@b4n is there something plugin specific to avoid saying to add those -Wno-XXX flags to CFLAGS env. var. and applying to all plugins?

Copy link
Member

Choose a reason for hiding this comment

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

--disable-extra-c-warnings for everyone. Nothing plugin-specific.

Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe I can add the #pragma to ignore it and ensure that file gets included in all the generated C sources?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants