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

cross compile: make it clearer what is being output #3629

Closed
wants to merge 1 commit into from

Conversation

rdp
Copy link
Contributor

@rdp rdp commented Dec 3, 2016

today it just outputs it without telling you what it is...

@ysbaddaden
Copy link
Contributor

Maybe output a few lines? To tell that the object file was created at path/foo.o, it should be copied to the target system (if needed) then run the following command to link the executable for example.

@asterite
Copy link
Member

We have this:

https://crystal-lang.org/docs/syntax_and_semantics/cross-compilation.html

I think that documents it more or less well. Adding just a line here won't work well... plus I think there's a lot more to do than just copy an .o, as you might need to install libs, maybe other stuff too.

Should we close this or merge this? (once rebased)

@RX14
Copy link
Contributor

RX14 commented Apr 28, 2018

i vote close - at least this way you can actually grab the cross-compile command programatically and run it somewhere. With this PR you'd have to strip out the message

@RX14 RX14 closed this Apr 28, 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.

None yet

4 participants