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

[JENKINS-50969] core site build is broken, needed for core taglibs reference #3408

Merged
merged 2 commits into from
May 5, 2018

Conversation

batmat
Copy link
Member

@batmat batmat commented Apr 26, 2018

The special old module for doxia seems not needed.
I suspect that it is because since 2012 when it was released, the Markdown
support got done in standard.

As this didn't get any update since 2012, IIUC this was pulling in some very old doxia versions, which was incompatible with maven-fluido-skin which I needed to bump from current 1.3.0 to 1.7 for https://github.com/jenkins-infra/core-taglibs-report-generator to work again.

See jenkins-infra/core-taglibs-report-generator#7 applied in practice.

See JENKINS-50969.

Proposed changelog entries

N/A internal change (maven-site-plugin only).

Submitter checklist

  • JIRA issue is well described
  • Changelog entry appropriate for the audience affected by the change (users or developer, depending on the change). Examples
    * Use the Internal: prefix if the change has no user-visible impact (API, test frameworks, etc.)
  • Appropriate autotests or explanation to why this change has no tests
  • For dependency updates: links to external changelogs and, if possible, full diffs

Desired reviewers

@daniel-beck

@batmat batmat added the work-in-progress The PR is under active development, not ready to the final review label Apr 26, 2018
@batmat batmat changed the title WIP [JENKINS-50969] core site build is broken, needed for core taglibs reference [JENKINS-50969] core site build is broken, needed for core taglibs reference Apr 26, 2018
@batmat batmat requested a review from daniel-beck April 26, 2018 14:27
@batmat batmat removed the work-in-progress The PR is under active development, not ready to the final review label Apr 26, 2018
@batmat batmat added needs-more-reviews Complex change, which would benefit from more eyes on-hold This pull request depends on another event/release, and it cannot be merged right now labels Apr 27, 2018
@batmat
Copy link
Member Author

batmat commented Apr 27, 2018

Putting this on-hold as I would like it to not be merged before #3410 is merged and released in a weekly. BUT it is ready for review.

…ference

The special old module for doxia seems not needed.
I suspect that it is because since 2012 when it was released, the Markdown
support got done in standard.

IIUC this was pulling in some very old doxia versions, which was
incompatible with maven-fluido-skin which I needed to bump from current
1.3.0 to 1.7 for https://github.com/jenkins-infra/core-taglibs-report-generator
to work again.
pom.xml Outdated
@@ -28,7 +28,7 @@ THE SOFTWARE.
<parent>
<groupId>org.jenkins-ci</groupId>
<artifactId>jenkins</artifactId>
<version>1.41</version>
<version>1.42</version>
Copy link
Member Author

@batmat batmat Apr 27, 2018

Choose a reason for hiding this comment

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

Bumps maven-site-plugin from 3.6 to 3.7. See jenkinsci/pom#21

@batmat batmat removed the on-hold This pull request depends on another event/release, and it cannot be merged right now label Apr 30, 2018
@batmat
Copy link
Member Author

batmat commented Apr 30, 2018

2.119 release is out, unon-holded

@oleg-nenashev
Copy link
Member

Parent POM update has been already done by @jglick in #3394
Resolved the merge conflict

@oleg-nenashev oleg-nenashev added ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback and removed needs-more-reviews Complex change, which would benefit from more eyes labels May 4, 2018
@batmat
Copy link
Member Author

batmat commented May 4, 2018

Thanks a lot Oleg.

@oleg-nenashev oleg-nenashev merged commit 4278461 into jenkinsci:master May 5, 2018
@batmat batmat deleted the JENKINS-50969 branch May 5, 2018 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants