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

Switch to system libjq #43

Closed
wants to merge 3 commits into from
Closed

Switch to system libjq #43

wants to merge 3 commits into from

Conversation

jeroen
Copy link
Member

@jeroen jeroen commented Mar 30, 2016

Removes the bundled jq and links to system libjq instead. Currently this works on Windows, OSX and Fedora, but not Debian/Ubuntu because libjq-dev is not yet available there. Hopefully it will become available soon. You can leave this PR open until then...

@sckott sckott added this to the v0.2.1 milestone Mar 30, 2016
@sckott
Copy link
Collaborator

sckott commented Apr 10, 2016

@jeroenooms where can I look to see if libjq-dev is on Debian yet?

@jeroen
Copy link
Member Author

jeroen commented Apr 10, 2016

I don't think so yet: https://packages.debian.org/sid/jq. @else ?

@elsbrock
Copy link

Sorry for the late reply guys, I'll try to work on getting libjq into Debian on the weekend.

@sckott
Copy link
Collaborator

sckott commented Apr 14, 2016

no worries, & thanks for the help

@sckott
Copy link
Collaborator

sckott commented Apr 17, 2016

i think this https://packages.debian.org/search?keywords=jq suggests we are ready to go? @jeroenooms ?

@elsbrock
Copy link

Nope, that package is still missing libjq, which is what I'm working on getting into Debian.

@sckott
Copy link
Collaborator

sckott commented Apr 17, 2016

woops, thanks for clarifying! (i'm not familiar with debian pkgs)

@jeroen
Copy link
Member Author

jeroen commented Apr 21, 2016

@else would it help if I have a first pass at this, and give you a working debian package for you to review/tweak before submitting?

@sckott sckott modified the milestones: v0.2.4, v0.2.1 Apr 21, 2016
@sckott sckott mentioned this pull request Jul 27, 2016
@sckott sckott removed this from the v0.2.6 milestone Sep 13, 2017
@sckott sckott mentioned this pull request Sep 17, 2017
@jeroen
Copy link
Member Author

jeroen commented Sep 18, 2017

Closing in favor of #59

@jeroen jeroen closed this Sep 18, 2017
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