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

Need to surface errors with static code analysis to make understanding problems easier #306

Open
jwmatthews opened this issue Jan 28, 2025 · 0 comments
Milestone

Comments

@jwmatthews
Copy link
Member

I ran an analysis of an old application: https://github.com/jboss-developer/ticket-monster/tree/2.7.0.Final-with-tutorials/demo

This yielded ONLY issues on the pom.xml, we did not understand why.
As in, we assumed that there would be issues on the *.java files, but this was not seen.

To debug, I ran mvn compile
Saw it fail with: https://gist.github.com/jwmatthews/e0ecc7eff323c063fcd064942f0568d3/edit

[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[ERROR] Non-resolvable import POM: The following artifacts could not be resolved: org.jboss.bom.wfk:jboss-javaee-6.0-with-tools:pom:2.7.0-redhat-1 (absent): org.jboss.bom.wfk:jboss-javaee-6.0-with-tools:pom:2.7.0-redhat-1 was not found in https://maven.repository.redhat.com/techpreview/all/ during a previous attempt. This failure was cached in the local repository and resolution is not reattempted until the update interval of redhat-techpreview-all-repository has elapsed or updates are forced @ line 85, column 25

After some digging in, I learned that the first issue is that the maven registry URL is invalid.
See: https://github.com/jboss-developer/ticket-monster/blob/2.7.0.Final-with-tutorials/demo/pom.xml#L25

I made the below change to fix and go further


 $ git diff
diff --git a/demo/pom.xml b/demo/pom.xml
index 8c4bd0b..43c0c3a 100644
--- a/demo/pom.xml
+++ b/demo/pom.xml
@@ -22,7 +22,7 @@
     <repositories>
         <repository>
             <id>redhat-techpreview-all-repository</id>
-            <url>https://maven.repository.redhat.com/techpreview/all/</url>
+            <url>https://maven.repository.redhat.com/ga/</url>
             <releases>
                 <enabled>true</enabled>
             </releases>
@@ -35,7 +35,7 @@
     <pluginRepositories>
         <pluginRepository>
             <id>redhat-techpreview-all-repository</id>
-            <url>https://maven.repository.redhat.com/techpreview/all/</url>
+            <url>https://maven.repository.redhat.com/ga/</url>
             <releases>
                 <enabled>true</enabled>
             </releases>

@djzager djzager added this to the v0.1.0 milestone Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

2 participants