-
-
Notifications
You must be signed in to change notification settings - Fork 354
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
No Maven Central release #12
Comments
I could make a Pull Request with the setup we use for our projects. |
That would look similar to this: https://github.com/CubeEngine/MessageCompositor/blob/master/pom.xml |
Hi Phillip, Thanks for the issue. Going back to Maven Central is indeed in the roadmap of the project. How is authorization handled to publish a new version on Maven Central? --Martin |
They use their JIRA for registration and once you got approved you just put a section in your local maven configuration with your JIRA login information and the ID of your configured central repository in . More information about the process: |
http://gforge.inria.fr/frs/shownotes.php?release_id=8745
Why did I even make the pull request to fix everything up for a release? |
Hi Phillip, |
A new status on this? |
Hello Phillip, Yes, I push on Maven Central today. :) |
Thanks for the deployment in Maven Central. It seems that the jar http://search.maven.org/remotecontent?filepath=fr/inria/gforge/spoon/spoon-core/2.3/spoon-core-2.3.jar contains a log4j.properties file with the following line 10:
I'm a bit confused since the original file as stored in the git repo https://github.com/INRIA/spoon/blob/master/src/main/resources/log4j.properties contains instead: log4j.appender.R.File=${java.io.tmpdir}/spoon-log.log Practically, when running Spoon 2.3 fetched from Maven central, I bump into the following exception: log4j:ERROR setFile(null,true) call failed. which is obviously evident given the log4j.properties. Wat do you think? Lionel. |
Well someone forgot to disable resource filtering... maven will replace ${...} macros and the velocity context to do this contains all java properties. |
Hi, I discussed the problem with @monperrus and we modified the log4j.properties in the commit. c7af8e2. I'm repushing on maven now. You will see it at the end of this morning or in the afternoon. |
You might want to check the commit comments first |
Yes, I saw them after my message. |
Well, I made a PR #55 to improve pom.xml to disable filtering on the log4j.properties file. If it is ok for all, we merge it and I re-push on Maven Central. |
I'm re-pushing on Maven Central now. It should appear on the website in some hours. |
Why didn't you use the release plugin?? |
I used maven release plugin. |
Well the release commits in git are missing as well as the tag. This leads to the problem the a) the source code used to build the release in maven central is not actually available and b) the pom.xml in this repo has the wrong version as it should be 2.3.2-SNAPSHOT or 2.4.0-SNAPSHOT |
Ah yes, this is "normal". We don't release on this Git project but on another one. @monperrus must to push commits and tag of the release plugin from the another git projet to this GitHub project (because, I can't myself, I don't have these privileges). |
Tag is now on Github. |
Just an idea: It might be good to automate the mirroring from the internel repository to github. I once did that with a small post-receive-hook and a deploy key. Anyway: This issue is now finally closed, I hope this will continue in the future. |
I have currently the problem, that I can't deploy new versions of a project of mine to the Maven Central Repository, because it recently started using spoon. The problem is, that spoon in its current version is not available in Maven Central, but to be able to deploy an artifact all of its dependencies must be available in Maven Central themselves.
Our workaround will be to deploy a forked version of spoon with a changed pom.xml until the official version is available.
The text was updated successfully, but these errors were encountered: