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

henplus: use :java requirement #806

Closed
wants to merge 1 commit into from

Conversation

ilovezfs
Copy link
Contributor

@ilovezfs ilovezfs commented May 3, 2016

No description provided.

@DomT4
Copy link
Member

DomT4 commented May 4, 2016

This won't test on CI because one of the dependencies wants Java 1.6, somehow.

Personally given the need for an ancient Java I'd just been tempted to kill this and libreadline-java but if you want to keep it I don't mind.

@ilovezfs
Copy link
Contributor Author

ilovezfs commented May 4, 2016

@DomT4 I've fixed the libreadline-java issue and reported it upstream: #837

In addition, henplus upstream has agreed to create a new tag: neurolabs/henplus#41

So I don't think either need be removed.

@DomT4
Copy link
Member

DomT4 commented May 4, 2016

Alright, as long as you're working on it. Anything using Java 1.6 these days frankly seems unsupported by upstream.

@ilovezfs
Copy link
Contributor Author

ilovezfs commented May 4, 2016

@DomT4 🚢?

@DomT4
Copy link
Member

DomT4 commented May 5, 2016

Seems reasonable 👍

@ilovezfs ilovezfs closed this in e611874 May 5, 2016
@ilovezfs ilovezfs deleted the henplus-trigger-ci branch June 3, 2016 11:07
@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants