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

Publish boopickle for Scala.js 1.0.0 #117

Closed
strelec opened this issue Feb 11, 2020 · 14 comments
Closed

Publish boopickle for Scala.js 1.0.0 #117

strelec opened this issue Feb 11, 2020 · 14 comments

Comments

@strelec
Copy link

strelec commented Feb 11, 2020

Good news, scala.js 1.0.0 has been released. It would be good to have boopickle published for it, for last few releases of boopickle.

@mathieuleclaire
Copy link
Collaborator

The PRs #118 and #121 and the CI fix commit do this for scala 2.12 and 2.13 JS and JVM.. There is only a final CI issue when deleting folders at the end; I don't no why. Otherwise all tests passed ! So a release can be done very soon !

rm: missing operand
Try 'rm --help' for more information
The command "find $HOME/.ivy2 -name "ivydata-*.properties" | xargs rm" exited with 123.

@mathieuleclaire
Copy link
Collaborator

f013b9b fixes the CI issue. So that we are now for releasing @ochrons

@mathieuleclaire
Copy link
Collaborator

I would really need to have a new version publshed with scalajs 1.0.0. Would you release it @ochrons ?

@ochrons
Copy link
Collaborator

ochrons commented May 12, 2020

Now published. Had a little bit of problem with ScalaDoc and Java11 (scala/scala#8663)

@ochrons ochrons closed this as completed May 12, 2020
@mathieuleclaire
Copy link
Collaborator

Excellent, thanks !

@vic
Copy link

vic commented May 12, 2020

@ochrons, thanks for publishing boopickle for sjs1.

I'm working on updating cornerman/chameleon#53 to run with boopickle 1.3.2. However, I'm getting the following error:

[error] java.io.IOException: Failed to deserialize /v/Library/Caches/Coursier/v1/https/repo1.maven.org/maven2/io/suzaku/boopickle_sjs1_2.12/1.3.2/boopickle_sjs1_2.12-1.3.2.jar:/boopickle/BasicPicklers$$anon$6.sjsir
[error] Caused by: java.lang.IllegalArgumentException: malformed version: 0.6.17

but we are using sjs 1.0.1, full error:
https://gist.github.com/vic/e4a05a73eeba91cf5cd1bcc4ed6c8069#file-gistfile1-txt-L32

Perhaps the boopickle artifact still contains .sjsir files correspoding to the scalajs 0.6.17?

Or perhaps, am I missing something?

Cheers :)

@strelec
Copy link
Author

strelec commented May 12, 2020

Even after clearing all caches. I can reproduce this issue.

@ochrons
Copy link
Collaborator

ochrons commented May 12, 2020

Perhaps something went wrong in the publishing process and it didn't rebuild using SJS1 but used some old version. Can you try with Scala 2.13, as I did thorough cleaning before publishing that version due to the aforementioned scaladoc issue.

@strelec
Copy link
Author

strelec commented May 12, 2020

Otto, you are correct.

The project for Scala 2.13.1 compiles successfully.

@ochrons
Copy link
Collaborator

ochrons commented May 12, 2020

I guess it's time to release 1.3.3 then to fix this 😅

@slandelle
Copy link
Contributor

@ochrons Beware that if you compile with Java 11, there's a chance you're breaking Java 8 compat. If your code is calling flip on a ByteBuffer, you'll be linking to the new override that doesn't exist on Java 8 (that only has Buffer#flip). You should be using Java 8 while releasing.

@ochrons
Copy link
Collaborator

ochrons commented May 12, 2020

Oh, good to know! I wonder if that could be somehow checked in the SBT build to prevent mistakes.

@slandelle
Copy link
Contributor

Actually, you could easily work around this in your case.
It seems you only have one single flip call in your whole main code.
You could have a helper method such as def flip(bb: Buffer): Unit = bb.flip() that would make sure you're linking to Buffer#flip and not ByteBuffer#flip.

@ioleo
Copy link

ioleo commented May 17, 2020

@strelec I've had a similar issue (sjs 0.6 leftovers) with publishLocal on zio/zio, when we switched to SJS 1.0. sbt clean was not enough. Manually deleting the whole target directory helped.

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

No branches or pull requests

6 participants