-
Notifications
You must be signed in to change notification settings - Fork 443
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
Uninformative exception when head
called on empty list
#46
Comments
BTW - I'm not 100% certain why you saw this. It means for some reason the tar file was not created, but the process returned success. After the next release of sbt-native-packager, I'd love to here feedback on what the new error message states. |
Sure. I'm not sure either. I was trying to create my first sbt packager package and I was having a lot of trouble. I was going to get it to work and improve the documentation, but then I ran into this. |
Anything I can do to help, please let me know. Perhaps if you had an example project with the problem, so I can try it out? |
Josh, I can give you a GitHub url to sample projects. Beware it has a test case with significant memory resources (this is a pain point for me). I'm probably doing a lot wrong, but I spent some time earlier with the available documentation.
If I remove the problematic line, the next error I get is:
https://github.com/knowitall/openie/tree/package1
|
Ah, so this is definitely a "documentation is poor" error. Specifically, here's a build.sbt that will work for you:
The changes are:
I'm updating the docs now! |
Cool. I saw both |
Yep. One is a carry-over from a rename. Updated the README.md with need-to-know info for now. |
https://github.com/sbt/sbt-native-packager/blob/master/src/main/scala/com/typesafe/sbt/packager/universal/Archives.scala#L126
I'm not sure what the cause is. We probably want a
.headOption.getOrElse(throw)
The text was updated successfully, but these errors were encountered: