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

builds of PR to R_29_Maintenance branch fail #885

Closed
the-snowwhite opened this issue Nov 11, 2023 · 11 comments
Closed

builds of PR to R_29_Maintenance branch fail #885

the-snowwhite opened this issue Nov 11, 2023 · 11 comments

Comments

@the-snowwhite
Copy link
Contributor

the-snowwhite commented Nov 11, 2023

Describe the bug
builds of PR to R_29_Maintenance branch fails with the error message:

[ERROR] Failed to execute goal org.eclipse.tycho.extras:tycho-p2-extras-plugin:4.0.2:compare-version-with-baselines
(compare-attached-artifacts-with-release) on project org.eclipse.swt: Only qualifier changed for 
(org.eclipse.swt/3.124.100.v20231111-0040). Expected to have bigger x.y.z than what is available in baseline 
(3.124.100.v20230825-1346)

running the command:

mvn clean verify --batch-mode -DforkCount=0 -Dcompare-version-with-baselines.skip=false -Dmaven.compiler.failOnWarning=true -Dmaven.test.failure.ignore=true -Dmaven.test.error.ignore=true

To Reproduce
A standalone SWT snippet to reproduce the behavior (preferable)
(or)
Steps to reproduce the behavior.
Perform a PR to R_29_maintenance branch

Expected behavior
The build succedes

Environment:

https://ci.eclipse.org/releng/job/eclipse.platform.swt/view/change-requests/job/PR-881/1/consoleFull

Workaround (or) Additional context
#858

Update:
The purpose of fixing this issue is

1.being able to enable successful PR builds on any branch
2. Fetching the build artifacts.
3. Closing the PR without a merge.

@merks
Copy link
Contributor

merks commented Nov 11, 2023

You must increase the version in the MANIFEST.MF. For the maintenance stream it would be add 1 to the third segment for this change.

@the-snowwhite
Copy link
Contributor Author

the-snowwhite commented Nov 11, 2023

I can find 2 MANIFEST.MF files in
/home/mib/git/eclipse.platform.swt/bundles/org.eclipse.swt

/home/mib/git/eclipse.platform.swt/bundles/org.eclipse.swt/target/MANIFEST.MF

/home/mib/git/eclipse.platform.swt/bundles/org.eclipse.swt/META-INF/MANIFEST.MF

only

/home/mib/git/eclipse.platform.swt/bundles/org.eclipse.swt/target/MANIFEST.MF

contains: 1346
So that must be the correct one.

update:
Correction I only have access to commit changes to
/home/mib/git/eclipse.platform.swt/bundles/org.eclipse.swt/META-INF/MANIFEST.MF

@merks
Thanks Do you know how often this number needs to be increased (like for each new PR) ?

the-snowwhite added a commit to the-snowwhite/eclipse.platform.swt that referenced this issue Nov 11, 2023
@merks
Copy link
Contributor

merks commented Nov 11, 2023

As you noticed, whatever is in the "target" folder is produced during the build. The version number needs to be increased if there is a change after the already-released version for the 4.29 stream was built. The build compares the already-released bundle to the bundle produced by the build. If they are different, and there is no version change whatsoever, the build fails complaining about only the qualifier changing; the qualifier is computed by the timestamp of the newest commit in the project's source.

I'm not exactly sure why you are changing the 4.29 stream given the platform doesn't publish/product maintenance releases...

the-snowwhite added a commit to the-snowwhite/eclipse.platform.swt that referenced this issue Nov 11, 2023
@the-snowwhite
Copy link
Contributor Author

#886 (comment)

@the-snowwhite
Copy link
Contributor Author

#886 (comment)

@the-snowwhite
Copy link
Contributor Author

Seems like this issue is right now blocked by something that not is in the book (Yet).

The purpose of fixing this issue is:

1.being able to enable successful PR builds on any branch
2. Fetching the build artifacts.
3. 3. Closing the PR without a merge.
@mickaelistria
Is this correct ?

@the-snowwhite
Copy link
Contributor Author

@merks
I have opened a New feature request here:
#888

@the-snowwhite
Copy link
Contributor Author

As you noticed, whatever is in the "target" folder is produced during the build. The version number needs to be increased if there is a change after the already-released version for the 4.29 stream was built. The build compares the already-released bundle to the bundle produced by the build. If they are different, and there is no version change whatsoever, the build fails complaining about only the qualifier changing; the qualifier is computed by the timestamp of the newest commit in the project's source.

I'm not exactly sure why you are changing the 4.29 stream given the platform doesn't publish/product maintenance releases...

The purpose was not to change the 4.29 stream.
the purpose is being able to Fetch build artifacts from any branch on the repository for testing purposes.
Not merging anything,

@the-snowwhite
Copy link
Contributor Author

I'm not exactly sure why you are changing the 4.29 stream given the platform doesn't publish/product maintenance releases...
@merks
Yes I already know that please look at: #858

@merks
Copy link
Contributor

merks commented Nov 11, 2023

Yes, I've seen a great deal of email.

@mickaelistria
Copy link
Contributor

THe build is passing if there is no change in it; if you introduce a change, then you need to bump the version as it wasn't done since last 4.29.x release. This is explained in https://wiki.eclipse.org/Version_Numbering

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

No branches or pull requests

3 participants