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

build of packaged (2.24.0) version fails #1021

Open
rpodgorny opened this issue Nov 9, 2019 · 6 comments
Open

build of packaged (2.24.0) version fails #1021

rpodgorny opened this issue Nov 9, 2019 · 6 comments

Comments

@rpodgorny
Copy link

rpodgorny commented Nov 9, 2019

when i clone the git repo and run script/build everything builds fine.

but when i download the 2.24.0.tar.gz package and try to build that, it fails with:

Creating an optimized build, using build cache to speed up build.
Specifying --fast skips optimizations and takes less than two minutes.
### Building planck-cljs
Syntax error (IllegalArgumentException) compiling at (script/build.clj:30:1).
/tmp/pkgbuilds/planck/src/planck-2.23.0/planck-cljs/out/goog/i18n/datetimesymbols.js is not a relative path

(running this from /home/user/tmp/pkgbuilds/planck) ...so it seems like the /home/user part gets cut off and that causes the failure.

@mfikes
Copy link
Member

mfikes commented Nov 9, 2019

@rpodgorny Hrm. That's odd, and I can't repro.

I'm wondering how planck-2.23.0 appears in the path in your issue description (while the downloaded package is for 2.24.0.tar.gz)

@rpodgorny
Copy link
Author

ahh, sorry about that. i also tried with 2.23 just to be sure it's not a problem of specific version but rather a problem of (any?) "packaged" version... ...the output for 2.24 is just the same.

@mfikes
Copy link
Member

mfikes commented Nov 9, 2019

Cool. Are you on Linux? (I tried reproing on macOS).

Happy to take a patch if you see the issue. Otherwise need to sort out how to actually repro. :)

@rpodgorny
Copy link
Author

yes, i'm on arch linux.

for me, it's just:

wget https://github.com/planck-repl/planck/archive/2.24.0.tar.gz
tar xvf 2.24.0.tar.gz
cd planck-2.24.0
script/build

i'll try to investigate further...

@rpodgorny
Copy link
Author

so, after copying planck-cljs/deps.edn from git version the the unpacked, everything works...
...the diff is:

--- planck-2.24.0/planck-cljs/deps.edn  2019-07-26 03:23:11.000000000 +0200
+++ planck/planck-cljs/deps.edn 2019-11-09 17:57:54.256620034 +0100
@@ -1,6 +1,8 @@
 {:deps {org.clojure/clojurescript {:mvn/version "1.10.520"}
+        org.clojure/google-closure-library {:mvn/version "0.0-20190213-2033d5d9"}
         org.clojure/test.check {:mvn/version "0.10.0-alpha4"}
         com.cognitect/transit-cljs {:mvn/version "0.8.256"}
+        com.cognitect/transit-js {:mvn/version "0.8.861"}
         fipp {:mvn/version "0.6.18"}
         malabarba/lazy-map {:mvn/version "1.3"}
         cljsjs/long {:mvn/version "3.0.3-1"}}}

i still wonder how come it works for you... ;-)

@rpodgorny
Copy link
Author

fyi, this workaround also works:

sed 's_lib/closure_lib/closure/goog_' -i planck-cljs/script/build.clj

(related to https://clojure.atlassian.net/browse/CLJS-2152)

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

2 participants