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

Support successfully bootstrapping on Solaris (OpenIndiana). #1069

Merged
merged 1 commit into from
Apr 26, 2018

Conversation

lanurmi
Copy link
Contributor

@lanurmi lanurmi commented Apr 26, 2018

Some support already existed, but not enough. Also the "linkoptions" provided was incompatible with the Solaris ld.

Some support already existed, but not enough. Also the "linkoptions" provided was incompatible with the Solaris ld.
@samsinsane
Copy link
Member

Does this fix Solaris, or just OpenIndiana? I wouldn't want to approve a PR that breaks the official OS so that an open source derivative works instead. Are you able to provide any additional context to this for my peace of mind?

@starkos You appear to be the one who originally added Solaris support, but on behalf of someone else? Do you have access to a Solaris box?

@lanurmi
Copy link
Contributor Author

lanurmi commented Apr 26, 2018

Does this fix Solaris, or just OpenIndiana?

I hope both, but I don't currently have access to a genuine Solaris to verify that.

I wouldn't want to approve a PR that breaks the official OS so that an open source derivative works instead. Are you able to provide any additional context to this for my peace of mind?

The two linkoptions lines were the only explicit reference to solaris within the bootstrap configuration, so I suspect that current master would not build on a genuine Solaris either. The rest of the configuration would treat Solaris as either Linux or BSD.

As for the linkoptions, this post suggests that --export-dynamic is a feature of GNU ld only. (GNU ld may be available on a Solaris system, but it is not the default ld.)

@samsinsane
Copy link
Member

@lanurmi True, sorry about that! I didn't go back over your changes again after reading up on Solaris and OpenIndiana, I should have since you haven't made any changes that look like they would cause issues.

I'll wait for Jason to respond to see if he's got access to a genuine Solaris machine, otherwise, we can probably just merge this. Thanks for changes!

@starkos
Copy link
Member

starkos commented Apr 26, 2018

@starkos You appear to be the one who originally added Solaris support, but on behalf of someone else? Do you have access to a Solaris box?

Umm…wow, no. I haven't even seen a Solaris box this century, I'm pretty sure. The original Solaris support was contributed by someone else, back in the dark days before GitHub :)

@samsinsane samsinsane merged commit 1a415df into premake:master Apr 26, 2018
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.

3 participants