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

Inline Maven property incrementals.url for Maven 4 support #666

Merged
merged 1 commit into from
Jan 19, 2023

Conversation

froque
Copy link
Member

@froque froque commented Jan 9, 2023

Building plugins with Maven 4 (alpha) fails with:

'profiles.profile[consume-incrementals].repositories.repository.[incrementals].url' contains an expression but should be a constant. @ org.jenkins-ci.plugins:plugin:4.53

This has already been reported to Jenkins [1] and Maven [2] and was declared as an intentional change.

[1] https://issues.jenkins.io/browse/JENKINS-67878
[2] https://issues.apache.org/jira/browse/MNG-7420

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

Building plugins with Maven 4 (alpha) fails with:
> 'profiles.profile[consume-incrementals].repositories.repository.[incrementals].url' contains an expression but should be a constant. @ org.jenkins-ci.plugins:plugin:4.53

This has already been reported to Jenkins [1] and Maven [2] and was
declared as an intentional change.

[1] https://issues.jenkins.io/browse/JENKINS-67878
[2] https://issues.apache.org/jira/browse/MNG-7420
@NotMyFault
Copy link
Member

Did maven 4 get a rid of interpolation of URLs of any kind 🤔

Copy link
Member

@basil basil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If merged, this would render the documentation at https://github.com/jenkinsci/incrementals-tools#offline-testing obsolete, implying the need for a corresponding documentation update.

froque added a commit to froque/incrementals-tools that referenced this pull request Jan 10, 2023
This is due to Maven 4 support [1].

Building plugins with Maven 4 (alpha) fails with:
> 'profiles.profile[consume-incrementals].repositories.repository.[incrementals].url' contains an expression but should be a constant. @ org.jenkins-ci.plugins:plugin:4.53

This has already been reported to Jenkins [2] and Maven [3] and was
declared as an intentional change.

[1] jenkinsci/plugin-pom#666
[2] https://issues.jenkins.io/browse/JENKINS-67878
[3] https://issues.apache.org/jira/browse/MNG-7420
@froque
Copy link
Member Author

froque commented Jan 10, 2023

I have give it a go updating the instructions with jenkinsci/incrementals-tools#33

Copy link
Member

@basil basil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested the instructions you provided in the other PR, and they seem to work fine. Thanks!

@basil basil changed the title Inline Maven property incrementals.url for Maven 4 support Inline Maven property incrementals.url for Maven 4 support Jan 19, 2023
@basil basil merged commit 59317c2 into jenkinsci:master Jan 19, 2023
@froque froque deleted the maven-4_incrementals.url branch January 19, 2023 19:23
Copy link
Member

@jglick jglick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems fine. jenkinsci/pom would need a corresponding change if it did not already happen. So far as I recall, the only purpose of defining this in a property was to make it easier to test the Incrementals infrastructure, which is long since stable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants