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

Removing JavaC dependency in *.bat #65

Merged
merged 1 commit into from
Nov 7, 2013
Merged

Removing JavaC dependency in *.bat #65

merged 1 commit into from
Nov 7, 2013

Conversation

Jentsch
Copy link
Contributor

@Jentsch Jentsch commented Nov 7, 2013

Hi everyone,
just stumbled over the fact that the bat script requires the Java compiler, but never use it. (and the bash script never mention the javac.)

May I miss a situation where javac is required, otherwise it would be nice if this pull will be accepted.

Kind regards, Jentsch

javac is never used in batch script and a normal project should depend
on it during runtime.
@Jentsch
Copy link
Contributor Author

Jentsch commented Nov 7, 2013

Maybe the Oppan Redmond OR could be also removed…

@jsuereth
Copy link
Member

jsuereth commented Nov 7, 2013

Ah, thanks for the fix. That was leftover from using this for Activator/sbt.

jsuereth added a commit that referenced this pull request Nov 7, 2013
Removing JavaC dependency in *.bat
@jsuereth jsuereth merged commit 20180ea into sbt:master Nov 7, 2013
@Jentsch Jentsch deleted the RemovingJavacInBAT branch November 7, 2013 18:47
@Jentsch
Copy link
Contributor Author

Jentsch commented Nov 7, 2013

How long will it take until this path is deployed? (Or when could I stop to manually fix the bat file manually after every 'dist'?)

@jsuereth
Copy link
Member

jsuereth commented Nov 7, 2013

I can cut a patch release soon. I need to backport this into a branch, because the latest has diverged past binary compatibility. I'll see if I can get you a release tomorrow. If not, it'll be 2 weeks as I'm on travel.

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.

2 participants