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

feat(22.04): add archive curl, libcurl4, libgcrypt20, libgpg-error0, libnhttp2-14, librtmp1, libssh2-1, libzstd1, libpsl5 #107

Merged
merged 14 commits into from Feb 13, 2024

Conversation

ghost
Copy link

@ghost ghost commented Jan 31, 2024

Libpsl allows checking domains against the Public Suffix List. It can be used to avoid privacy-leaking 'super-cookies', 'super domain' certificates, for domain highlighting purposes sorting domain lists by site and more.

Refer below package list for dependencies list
https://packages.ubuntu.com/jammy/libs/libpsl5


Forward-port PRs

@ghost ghost closed this Jan 31, 2024
@ghost ghost reopened this Jan 31, 2024
@ghost
Copy link
Author

ghost commented Jan 31, 2024

@cjdcordeiro , @rebornplusplus ,@letFunny
May I know who to assign this PR for review?

@ghost ghost changed the base branch from ubuntu-22.04 to ubuntu-22.10 February 1, 2024 05:08
@ghost ghost changed the base branch from ubuntu-22.10 to ubuntu-22.04 February 1, 2024 05:09
@cjdcordeiro cjdcordeiro self-assigned this Feb 1, 2024
@cjdcordeiro
Copy link
Collaborator

Thanks for the PR @yashashwi . We'll review it shortly. In the meantime, you might want to update your branch with the latest changes.

Can you please also create the corresponding PRs for 23.10 and 24.04? Thanks :)

Copy link
Member

@rebornplusplus rebornplusplus 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 to me, thanks. Left a nitpick comment only.

slices/libpsl5.yaml Outdated Show resolved Hide resolved
@ghost
Copy link
Author

ghost commented Feb 1, 2024

@cjdcordeiro This change cannot directly go to 23.10 and 24.04, because we don't have libidn2-0_libs
- libunistring2_libs, in those upstreams.
Will raise PR in some time with all nested libraries till libpsl5 for 23.10 and 24.04.

Copy link
Collaborator

@cjdcordeiro cjdcordeiro left a comment

Choose a reason for hiding this comment

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

Cool. Thanks @yashashwi . We'll then hold this until those PRs are up, but it LGTM ;)

@cjdcordeiro cjdcordeiro added the Blocked Waiting for something external label Feb 1, 2024
@ghost
Copy link
Author

ghost commented Feb 1, 2024

@cjdcordeiro
I have raised the PR for ubuntu-23.10- [https://github.com//pull/107]
For Ubuntu:24.04 some of the slices like libunistring2 are not in the archive of noble. Therefore not raising the PR for 24.04

Refer below exception:
2024/02/01 15:51:09 Fetching index for ubuntu 24.04 noble-updates universe component...
error: slice package "libunistring2" missing from archive

@ghost ghost closed this Feb 1, 2024
@ghost ghost reopened this Feb 1, 2024
@rebornplusplus
Copy link
Member

@cjdcordeiro I have raised the PR for ubuntu-23.10- [https://github.com//pull/107] For Ubuntu:24.04 some of the slices like libunistring2 are not in the archive of noble. Therefore not raising the PR for 24.04

Refer below exception: 2024/02/01 15:51:09 Fetching index for ubuntu 24.04 noble-updates universe component... error: slice package "libunistring2" missing from archive

libpsl5 depends on libunistring5 actually.

@ghost
Copy link
Author

ghost commented Feb 2, 2024

@cjdcordeiro I have raised the PR for ubuntu-23.10- [https://github.com//pull/107] For Ubuntu:24.04 some of the slices like libunistring2 are not in the archive of noble. Therefore not raising the PR for 24.04
Refer below exception: 2024/02/01 15:51:09 Fetching index for ubuntu 24.04 noble-updates universe component... error: slice package "libunistring2" missing from archive

libpsl5 depends on libunistring5 actually.

@rebornplusplus yes it depends on that,
1.what does this exception mean - missing from archive. ?
2. Am I good to merge this PR for now, since this PR is for JAmmy.

@ghost ghost marked this pull request as draft February 2, 2024 18:43
@ghost ghost changed the title feat: add new archive for libpsl5 to ubuntu-22.04 release feat(22.04): add archive curl, libcurl4, libgcrypt20, libgpg-error0, libnhttp2-14, librtmp1, libssh2-1, libzstd1, libpsl5 Feb 2, 2024
@ghost ghost marked this pull request as ready for review February 2, 2024 18:44
@ghost
Copy link
Author

ghost commented Feb 2, 2024

The ultimate aim was to push curl bin, and since the PR was yet to be merged- I added complete package for Curl to execute.
Kindly re-review.
@cjdcordeiro @rebornplusplus @niemeyer

@ghost
Copy link
Author

ghost commented Feb 6, 2024

Thanks for the changes @yashashwi .

It's working well. Let's skip 24.04 if it cannot be done atm.

Once @rebornplusplus 's comments are addressed and 23.10's PR ready for review, this can be merged ;)

@cjdcordeiro I tried to raise the similar PR with curl for 23.10, but there are certain dependencies which does not have archive available for them in 23.10, Hence cannot raise the PR for 23.10.

I think PR #114 is sufficient for now. Lets merge PR 114 and this PR. Meanwhile is there any document that discusses how to update the archive, would like to update the same for 23.10 and 24.04

slices/libcurl4.yaml Outdated Show resolved Hide resolved
slices/libcurl4.yaml Show resolved Hide resolved
Copy link
Collaborator

@cjdcordeiro cjdcordeiro left a comment

Choose a reason for hiding this comment

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

ty! lgtm

@ghost ghost requested a review from cjdcordeiro February 6, 2024 15:29
@ghost
Copy link
Author

ghost commented Feb 6, 2024

accidently @cjdcordeiro review is requested again. No change in the PR.

Copy link
Collaborator

@cjdcordeiro cjdcordeiro left a comment

Choose a reason for hiding this comment

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

Np, but while reviewing #114 and the claim on incompatibility with 24.04 I realized that there are slices in this PR that can be ported to newer releases:

  • curl
  • libcurl4
  • libgpg-error0
  • libgcrypt20
  • libnghttp2-14
  • librtmp1
  • libssh2-1
  • libzstd1

NOTE: I know that the ultimate goal is to have curl, but if it helps to merge libpsl5 as soon as possible, feel free to move the above SDFs to a separate PR, keeping this PR focused on libpsl5 only

@cjdcordeiro cjdcordeiro added the Blocked Waiting for something external label Feb 6, 2024
@ghost
Copy link
Author

ghost commented Feb 6, 2024

Np, but while reviewing #114 and the claim on incompatibility with 24.04 I realized that there are slices in this PR that can be ported to newer releases:

  • curl
  • libcurl4
  • libgpg-error0
  • libgcrypt20
  • libnghttp2-14
  • librtmp1
  • libssh2-1
  • libzstd1

I will raise a separate PR for 23.10 and 24.04, this is having too much overlap, I will raise a fresh, dont worry. I have this in my task list. Also wanted to know what do we have to do to add a certain package avaiable for certain release?

@cjdcordeiro
Copy link
Collaborator

Cool ;) I ack the extra work, but it is a policy we need to enforce to make sure the DB of slices grows consistently across releases (to avoid inconsistent slice names and definitions for the same package in different releases)

what do we have to do to add a certain package avaiable for certain release?

Do you mean, adding a package to the upstream Ubuntu archives? If so, this is a distro question. Assuming you're talking about libunistring2, the best is to create a bug report in https://bugs.launchpad.net/ubuntu/+source/libunistring/+filebug

slices/libcurl4.yaml Outdated Show resolved Hide resolved
slices/libcurl4.yaml Outdated Show resolved Hide resolved
slices/libgpg-error0.yaml Outdated Show resolved Hide resolved
slices/libssh2-1.yaml Outdated Show resolved Hide resolved
slices/libssh2-1.yaml Outdated Show resolved Hide resolved
slices/libssh2-1.yaml Outdated Show resolved Hide resolved
slices/libzstd1.yaml Outdated Show resolved Hide resolved
slices/libzstd1.yaml Outdated Show resolved Hide resolved
slices/libzstd1.yaml Outdated Show resolved Hide resolved
@cjdcordeiro cjdcordeiro self-requested a review February 9, 2024 07:11
@ghost
Copy link
Author

ghost commented Feb 13, 2024

@rebornplusplus @cjdcordeiro
1.I have fixed all the open question in this PR.
2. Raised PR for noble #133
This PR is good for review.

Copy link
Collaborator

@cjdcordeiro cjdcordeiro left a comment

Choose a reason for hiding this comment

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

Thanks @yashashwi !

This LGTM.
Thanks for #133 . We also need one for 23.10, excluding libpsl5 because of the
libunistring2 issue.

@ghost
Copy link
Author

ghost commented Feb 13, 2024

Thanks @yashashwi !

This LGTM.
Thanks for #133 . We also need one for 23.10, excluding libpsl5 because of the
libunistring2 issue.

@cjdcordeiro for 23.10, I have raised already its already in the list of open PR.

Copy link
Member

@rebornplusplus rebornplusplus 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 to me now, thank you!

@rebornplusplus
Copy link
Member

Ya, I have updated the PR description with the list of forward-port PRs.

@cjdcordeiro
Copy link
Collaborator

Indeed, I missed it from the list of open PRs. all good!

@cjdcordeiro cjdcordeiro removed the Blocked Waiting for something external label Feb 13, 2024
@cjdcordeiro cjdcordeiro merged commit b989982 into canonical:ubuntu-22.04 Feb 13, 2024
gregory-schiano pushed a commit to gregory-schiano/chisel-releases that referenced this pull request Jun 22, 2024
…libnhttp2-14, librtmp1, libssh2-1, libzstd1, libpsl5 (canonical#107)
cjdcordeiro pushed a commit to ozanmakes/chisel-releases that referenced this pull request Sep 27, 2024
…libnhttp2-14, librtmp1, libssh2-1, libzstd1, libpsl5 (canonical#107)
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.

3 participants