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

Include IceCC's compilerwrapper binary #12

Open
aperezdc opened this issue Sep 20, 2017 · 1 comment
Open

Include IceCC's compilerwrapper binary #12

aperezdc opened this issue Sep 20, 2017 · 1 comment

Comments

@aperezdc
Copy link
Owner

From compilerwrapper.cpp:

/*
Older icecream versions assume the compiler is always GCC. This can
be fixed on the local side, but remote nodes would need icecream upgrade.
As a workaround icecc-create-env includes this wrapper binary in the environment
if clang is to be used as well, that will either call clang or the real gcc.
Which one depends on an extra argument added by icecream.
*/

While not strictly needed with newer version of IceCC, it would be good to include it in the same way as icecc-create-env does to ensure compatibility of the generated tarballs with older IceCC versions.

@aperezdc
Copy link
Owner Author

One way to go about this would be:

  • Optionally supporting a configuration directory at $XDG_CONFIG_HOME/popsicle. If there is a compilerwrapper symlink inside it, it will be assumed to point to the corresponding binary provided by IceCC.
  • If the configuration directory does not exist, or there is no compilerwrapper symlink, or it points to a non-existing file, then support for adding it into generated tarballs is disabled.

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

No branches or pull requests

1 participant