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

Don't write LIBARCHIVE.creationtime attribute in tar.gz archive entries #4427

Merged

Conversation

HannesWell
Copy link
Member

Fixes #2586

@HannesWell HannesWell added the backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change label Nov 10, 2024
Copy link
Member

@laeubi laeubi left a comment

Choose a reason for hiding this comment

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

There should be a config option to switch this behavior on/off

  • for Tycho 4.x it must be on by default
  • for Tycho 5.x we can decide what should be the default

@HannesWell
Copy link
Member Author

There should be a config option to switch this behavior on/off

* for Tycho 4.x it must be on by default

* for Tycho 5.x we can decide what should be the default

If that's necessary get consensus I can add an option, but AFAIK an option was not added when this attribute was introduced, was it?

@laeubi
Copy link
Member

laeubi commented Nov 10, 2024

when this attribute was introduced

This attribute was never (intentionally) introduced, we updated the library behind that and those maintainer choose for some reasons to support this "non standard" extension and add it by default. So from a compatibility point of view it only adds something new that can be ignored by a tool if desired.

No we have the other way round, that removing it might break use-cases that have relied on this, so I better like to preserve this. It would have been different if we would have never added it before but now it is there (even not added by intention) but disabling that by a config option is always possible of course.

@HannesWell
Copy link
Member Author

No we have the other way round, that removing it might break use-cases that have relied on this, so I better like to preserve this. It would have been different if we would have never added it before but now it is there (even not added by intention) but disabling that by a config option is always possible of course.

Ok, then lets do it this way.
But I suggest to, for convenience to disable it by default for tycho-5 and keep the property by default for Tycho-4.
If one day tar can handle or ignore that attribute and the version that has the fix is widely adopted we can re-add it again.

@HannesWell
Copy link
Member Author

@laeubi are you fine with this now?

@HannesWell HannesWell removed the backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change label Nov 10, 2024
@HannesWell
Copy link
Member Author

Copy link

github-actions bot commented Nov 10, 2024

Test Results

  603 files  ±0    603 suites  ±0   3h 58m 11s ⏱️ - 33m 11s
  431 tests ±0    423 ✅  - 1   7 💤 ±0  1 ❌ +1 
1 293 runs  ±0  1 270 ✅  - 1  22 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit d080159. ± Comparison against base commit 2e98ec9.

♻️ This comment has been updated with latest results.

laeubi
laeubi previously approved these changes Nov 10, 2024
Copy link
Member

@laeubi laeubi left a comment

Choose a reason for hiding this comment

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

Looks good!

@HannesWell HannesWell enabled auto-merge (rebase) November 10, 2024 19:14
@HannesWell HannesWell merged commit eef9e7f into eclipse-tycho:main Nov 10, 2024
12 of 15 checks passed
@HannesWell HannesWell deleted the remove-creationtime-attribute branch November 10, 2024 22:30
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

Successfully merging this pull request may close these issues.

Tycho creating tar files with extended header keyword 'LIBARCHIVE.creationtime'
2 participants