-
Notifications
You must be signed in to change notification settings - Fork 27
-
Notifications
You must be signed in to change notification settings - Fork 27
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
Clojars release 1.1.4 might be broken - completable-future is missing in my jar #52
Comments
I saw that 1.1.4 is quite old and completable-future is quite new code. |
Yeah, the issue is basically that I no longer have rights to that repo in Clojars after I left the company, so I tried to deploy it, and I failed. I probably need to fork the repo and set up a new Claypoole package (maybe just |
Yes we use it. |
If the java code does not change that often, it could be published as a separate library from the Clojure parts and use deps.edn (with git dependencies) for the clojure parts. |
@leon-barrett Do you have plans to fork the project and do that part? I am not affiliated with them but I would contribute some patches. |
deps.edn also supports preparation of a lib with Java sources in it now: |
Yeah, I wrote Clojure professionally from 2012-2017, but I haven't touched it in a while now. I'd love to hand this project off. (I contacted people at Climate to see about getting updated versions pushed, but I wouldn't hold my breath.) |
Okay, I filed issue clj-commons/meta#66 edit: but closed it in favor of clj-commons/meta#65 |
If they are not interested in maintaining it (and it seems this is the case) maybe they would like to transfer it so that other people can maintain it. Thanks for being open to the idea and active in contacting your former employees. |
Just let me know where and how to transfer and I'll move it. |
@jmelching Thanks! I think you'll transfer it to https://github.com/slipset , and then then they can transfer it on to the organization https://github.com/clj-commons/ . Discussion at clj-commons/meta#65 (comment) When slipset is ready, you can transfer it via instructions available at https://docs.github.com/en/repositories/creating-and-managing-repositories/transferring-a-repository . Thanks again! |
@jmelching I'll invite you to clj-commons, and you'll be able to do the transfer. |
@jmelching You should have gotten an invite. To transfer the repo, you need to the repo's "settings" tab and all the way down at the bottom, in the "Danger zone" there is a button "Transfer". Click that, and follow instructions. |
This is complete. https://github.com/clj-commons/claypoole |
Excellent! Thanks a lot |
I think this should be closed after the new release is done that includes the changes in the examples (completable-future). |
Signed-off-by: Eugen Stan <eugen.stan@netdava.com>
Should be fixed in release 1.2.2 on clojars (pending announcement) and available o latest branch. |
Hello,
I added this library to my project to test it out.
I'm using
com.climate/claypoole {:mvn/version "1.1.4"}
with deps.edn.I copied the examples and I got an error when I tried to run the piece of code with:
I got this in Calva:
Checked the jar file locally and found out the macro is missing.
File looks kind of old (July 2016) so maybe the build might be broken?
Perhaps some other fixes are also missing from latest release.
The text was updated successfully, but these errors were encountered: