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

Zyppng #1603

Draft
wants to merge 30 commits into
base: master
Choose a base branch
from
Draft

Zyppng #1603

wants to merge 30 commits into from

Conversation

jreidinger
Copy link
Contributor

@jreidinger jreidinger commented Sep 13, 2024

Problem

Short description of the original problem.

Solution

Short description of the fix.

Testing

  • Added a new unit test
  • Tested manually

Screenshots

Example output from my distrobox:

System repos:
Some("openSUSE-Tumbleweed-Debug")
Some("openSUSE-Tumbleweed-Non-Oss")
Some("Open H.264 Codec (openSUSE Tumbleweed)")
Some("openSUSE-Tumbleweed-Oss")
Some("openSUSE-Tumbleweed-Source")
Some("openSUSE-Tumbleweed-Update")

Host repos:
Some("repo-non-free (20240910)")
Some("The Latest YaST Packages")
Some("Visual Studio Code")
Some("Rust (15.5)")
Some("nVidia Graphics Drivers")
Some("fedora40")
Some("google-chrome")
Some("X11:Utilities")
Some("repo-non-oss (20240910)")
Some("repo-non-oss-debug (20240910)")
Some("repo-openh264 (20240910)")
Some("repo-oss (20240910)")
Some("repo-oss-debug (20240910)")
Some("repo-oss-source (20240910)")
Some("update-backports (20240910)")
Some("update-backports-debug (20240910)")
Some("update-non-oss (20240910)")
Some("update-non-oss-debug (20240910)")
Some("update-oss (20240910)")
Some("update-oss-debug (20240910)")
Some("update-sle (20240910)")
Some("update-sle-debug (20240910)")
Some("Libdvdcss Repository")
Some("packman")
Some("Source Repository (Non-OSS)")

@mvidner
Copy link
Contributor

mvidner commented Sep 17, 2024

FWIW, when I build this branch in the /testing_using_container.sh setup,

  1. Compilation fails unless I zypper install glib2-devel
  2. Then it fails because it cannot link -lzypp-glib ... so I will need to build a modified libzypp for that.

@mvidner
Copy link
Contributor

mvidner commented Sep 20, 2024

Thanks for the readme and the docs!

Right now the PR is a Draft targeting master. Do you have a plan to make it mergeable soon? Or is that unknown and to be determined in the review meeting and/or with Zypp team?

@jreidinger
Copy link
Contributor Author

@mvidner plan is not to merge it now as it is build as part of cargo build in rust directory. So it is just draft to demonstrate it and start of discussion. Maybe we will need to hide it with some feature flag.

@mvidner
Copy link
Contributor

mvidner commented Sep 20, 2024

@jreidinger I have adjusted the setup script, to work in a agama-testing:latest podman container.

It still fails for me with distrobox + 2024-09-17 Tumbleweed (not root, sudo missing), I wonder why it works for you

@mvidner
Copy link
Contributor

mvidner commented Sep 20, 2024

BTW we may want to set up the new zypp in /usr/local because after it gets installed, it interferes with package management 🤣

# zypper se nosuchpackage
[...]
No matching items found.
[...]
corrupted size vs. prev_size in fastbins
^C��
    &V^C�EFV^C�EFV^\Quit (core dumped)

It could be as easy as
cmake -D CMAKE_INSTALL_PREFIX=/usr/local ...

mvidner and others added 5 commits September 23, 2024 16:53
It still fails but explains why
Before, it just failed because of set -e and missing sudo
Makefile.cvs would do it in build/ and then here in ./
Call it once, in build/
mvidner pushed a commit to agama-project/zypp-rs that referenced this pull request Sep 26, 2024
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.

2 participants