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

Implement resolve-system-dependencies in c++ #989

Closed
shlevy opened this issue Jul 21, 2016 · 11 comments
Closed

Implement resolve-system-dependencies in c++ #989

shlevy opened this issue Jul 21, 2016 · 11 comments
Labels
macos Nix on macOS, aka OS X, aka darwin
Milestone

Comments

@shlevy
Copy link
Member

shlevy commented Jul 21, 2016

This is needed for the perl-to-c++ work, but I don't have access to a darwin box to actually run it. @pikajude can you take this one? I'll help out however I can.

@shlevy shlevy added this to the perl-to-c++ milestone Jul 21, 2016
@shlevy
Copy link
Member Author

shlevy commented Jul 21, 2016

(note that non-darwin can be perl-free without this due to 3c68a66)

@shlevy
Copy link
Member Author

shlevy commented Jul 29, 2016

@pikajude ping

@domenkozar
Copy link
Member

domenkozar commented Aug 3, 2016

@shlevy don't we need to port the rest of the scripts to C++? Otherwise the closure still includes Perl.

@shlevy
Copy link
Member Author

shlevy commented Aug 3, 2016

Yes, this is just about the resolve-system-dependencies one which I can't do because I don't have a darwin box.

@domenkozar
Copy link
Member

@shlevy if you don't manage to get help from @pikajude, I'm sure you could get access to mac-5 from the build slaves to implement this.

@domenkozar domenkozar added the macos Nix on macOS, aka OS X, aka darwin label Aug 8, 2016
@pikajude
Copy link
Contributor

pikajude commented Aug 8, 2016

Hi everyone, sorry I just saw the emails for this thread (Github sends me a lot of emails). I'll try to work on this today

@domenkozar
Copy link
Member

Note that it's worth fixing #941 while rewriting this.

@shlevy
Copy link
Member Author

shlevy commented Aug 11, 2016

@pikajude status on this? It's the last thing in the perl removal...

@pikajude
Copy link
Contributor

I'll have very limited time over the next few days. I'll try to get this knocked out but it may not be done till next week

On Aug 11, 2016, at 8:02 AM, Shea Levy notifications@github.com wrote:

@pikajude status on this? It's the last thing in the perl removal...


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@shlevy
Copy link
Member Author

shlevy commented Aug 11, 2016

Thanks! No rush, just wanted to know if I should pick it up.

@domenkozar
Copy link
Member

This has been merged to master in #1030

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
macos Nix on macOS, aka OS X, aka darwin
Projects
None yet
Development

No branches or pull requests

3 participants