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

Dependency version bumps in advance of 3.0-RC1. #1773

Merged
merged 10 commits into from
Mar 12, 2016
Merged

Conversation

farmdawgnation
Copy link
Member

Okay here we go!

These are version bumps I want to submit for 3.0-RC1. They spawn a few deprecation warnings in the compile cycle so it may be worth taking care of those before merging.

AFAICT these are the highest version bumps I can make without breaking the compile. I also bumped the version of Scala while I was at it.

@fmpwizard
Copy link
Member

Looks like I didn't push to master the bump of common file upload, let me do that later tonight before this or gets in.

@fmpwizard
Copy link
Member

disregard my comment, there have been other commits to master after I made the bump internally, and I only changed the version number so merging this one in is just fine.

@karma4u101
Copy link
Member

Hi Matt
I notice you did not bump the servlet_api version to 3.0 is there any particular reason for it or just a oversight ?

@farmdawgnation
Copy link
Member Author

javax.servlet.servlet-api looks to only have a 3.0 alpha. Has package identifier changed for a 3.0 final?

@eltimn
Copy link
Member

eltimn commented Feb 24, 2016

It looks like maybe it was moved to: http://mvnrepository.com/artifact/javax.servlet/javax.servlet-api

@eltimn
Copy link
Member

eltimn commented Feb 24, 2016

Should we be using 3.1.0?

@karma4u101
Copy link
Member

Yea 3.1 seem like a good candidate (for JavaEE 7) if it dose not break things, what do you all think ?

@farmdawgnation
Copy link
Member Author

I think we should take advantage of a major version release and do it. Worst case scenario is we bump it back down during RC because it causes issues.

@karma4u101
Copy link
Member

Good point, 2.5 is more than 10 years old (September 2005). 3.0 is from December 2009 and 3.1 is from May 2013 so it would be a huge time leap forward ;)

@joescii
Copy link
Contributor

joescii commented Feb 27, 2016

Regarding servlet, we should also bump our starter kit's web.xml. That's what caused the heroku issues I was having.

@farmdawgnation
Copy link
Member Author

I'm working on the Servlet bump now.

@farmdawgnation
Copy link
Member Author

Uh there are quite a lot of compile errors that resulted from this in the test kit.

And quite a lot of:

class MockServletContext needs to be abstract, since: it has 26 unimplemented members.

😞

@farmdawgnation
Copy link
Member Author

Alright, I fixed all of the type errors in return types. It looks like the new Servlet API using more restrictive types (yay!) but I had to stop before I could provide sensible implementations for the other stuff since I'm sick today. 😷

If this doesn't break any tests, then I'll look at getting on a bit later and actually finishing out the implementations we need.

@Shadowfiend
Copy link
Member

(I'm hacking away at scalaz issues… More to come.)

There were about 16 million compile and execution errors... They're fixed.
Mostly. Sort of. Hopefully.
specs2 can occasionally run in a context where our current stack trace
inspection doesn't *quite* work.
I think specs2 used to provide a default Scala execution context, but it no
longer does so we had to import our own.
specs2 stopped supporting mixing MatchersImplicits directly into something that
extends Scope, but we can just import the implicits via the MatchersImplicits
singleton.
@Shadowfiend
Copy link
Member

Just pushed fixes for Scalaz issues, which include a bump to specs2 3.7 (latest) and Scalaz to 7.2.0 (latest). Had to tweak a few things along the way, including correct run mode detection for the latest specs2 and some implicit- and scalacheck-related issues.

@Shadowfiend
Copy link
Member

Ok, I'm down to merge this. I think we should mention when we cut RC1 that support for Servlet 3.1 is partial, and that we'd like to hear feedback on what features that we haven't implemented are important for folks.

If there are no objections, I'll merge tomorrow.

@fmpwizard
Copy link
Member

+1

On Thu, Mar 10, 2016 at 1:46 PM, Antonio Salazar Cardozo <
notifications@github.com> wrote:

Ok, I'm down to merge this. I think we should mention when we cut RC1 that
support for Servlet 3.1 is partial, and that we'd like to hear feedback on
what features that we haven't implemented are important for folks.

If there are no objections, I'll merge tomorrow.


Reply to this email directly or view it on GitHub
#1773 (comment).

Diego Medina
Lift/Scala Consultant
diego@fmpwizard.com
http://blog.fmpwizard.com/

@Shadowfiend
Copy link
Member

Alllllll righty. In we go! 👍

Shadowfiend added a commit that referenced this pull request Mar 12, 2016
Dependency version bumps in advance of 3.0-RC1.

These are version bumps I want to submit for 3.0-RC1. They spawn a few deprecation warnings in the compile cycle so it may be worth taking care of those before merging.

AFAICT these are the highest version bumps I can make without breaking the compile. I also bumped the version of Scala while I was at it.
@Shadowfiend Shadowfiend merged commit d85146f into master Mar 12, 2016
@Shadowfiend Shadowfiend deleted the raise-the-roof branch March 12, 2016 14:39
@karma4u101
Copy link
Member

@farmdawgnation @Shadowfiend
I just noticed a test regression but I am not totally sure this PR is the one affecting some tests but it seem like it could be (sry for being lazy and not checking it thoroughly) so maybe this been tackled elsewhere ?

I notice both in my own test and lifts FlexMenuBuilderSpec test that using ".... withSFor(url) in {...}" seems to silently drop the entire test.

The current output from Lifts FlexMenuBuilder Specification, looks like the following, it has 0 example(s) and that's a bit strange it should have 4. So dose WebSpecSpec ,HtmlPropertiesSpec and RestHelperSpec

[info] FlexMenuBuilder Specification
[info]
[info] FlexMenuBuilder should
[info]
[info]
[info] Total for specification FlexMenuBuilder Specification
[info] Finished in 0 ms
[info] 0 example, 0 failure, 0 error

@Shadowfiend
Copy link
Member

Looking into this… Maybe the way that I fixed WebSpec for the latest specs2 isn't quite right.

@Shadowfiend Shadowfiend mentioned this pull request Mar 15, 2016
@karma4u101
Copy link
Member

Awesome !

@Shadowfiend Shadowfiend modified the milestone: 3.0-RC1 Mar 20, 2016
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.

6 participants