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

[22.04] Add slices for libgnutls, libsoup and glib-networking #49

Open
wants to merge 5 commits into
base: ubuntu-22.04
Choose a base branch
from

Conversation

ozanmakes
Copy link

This PR adds various networking libraries and their dependencies.

@ozanmakes ozanmakes changed the title Add libgnutls, libsoup and glib-networking [22.04] Add slices for libgnutls, libsoup and glib-networking Aug 8, 2023
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 for the contribution @osener . This is a very welcome addition :)

Although functional, I've left a few requests that relate to the design of slices. The overall message is that we should be as conservative as possible when it comes to the use of globs. With these changes, it should be ready to be merged.

slices/glib-networking.yaml Outdated Show resolved Hide resolved
slices/gsettings-desktop-schemas.yaml Outdated Show resolved Hide resolved
slices/gsettings-desktop-schemas.yaml Outdated Show resolved Hide resolved
slices/gsettings-desktop-schemas.yaml Outdated Show resolved Hide resolved
slices/gsettings-desktop-schemas.yaml Outdated Show resolved Hide resolved
slices/libsoup2.4-1.yaml Show resolved Hide resolved
slices/libsoup2.4-1.yaml Outdated Show resolved Hide resolved
slices/libsoup2.4-1.yaml Outdated Show resolved Hide resolved
slices/libsoup2.4-1.yaml Outdated Show resolved Hide resolved
slices:
data:
contents:
/usr/share/publicsuffix/*:
Copy link
Collaborator

Choose a reason for hiding this comment

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

this path seems to be specific to this pkg, so the glob is a bit safer to use here. Nonetheless, given that there are only 3 files within, it is safer to just list them explicitly :)

@cjdcordeiro cjdcordeiro self-assigned this Aug 18, 2023
@rebornplusplus rebornplusplus added the decaying It's been a while, close or act on it label Aug 1, 2024
@rebornplusplus
Copy link
Member

Hi @ozanmakes, it has been some time since the last activity here. The PR could use a rebase/merge since there are some conflicts already. There are some changes requested as well. Will you please let me know if you have some time to go over the comments and update the PR? Thank you!

Copy link

github-actions bot commented Sep 26, 2024

Diff of dependencies:

slices/glib-networking-services.yaml
@@ -1,4 +0,0 @@
-glib-networking-common
-libc6
-libglib2.0-0
-libproxy1v5
slices/gsettings-desktop-schemas.yaml
@@ -1,3 +0,0 @@
-dconf-gsettings-backend
-gsettings-backend
-session-migration

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
decaying It's been a while, close or act on it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants