Skip to content
This repository has been archived by the owner on Oct 3, 2020. It is now read-only.

Fix RPM dependencies #32

Closed
wants to merge 2 commits into from
Closed

Fix RPM dependencies #32

wants to merge 2 commits into from

Conversation

TheAssassin
Copy link
Member

No description provided.

@TheAssassin
Copy link
Member Author

@fusion809
Copy link

fusion809 commented Nov 21, 2018

I built it locally with this patch applied, with wget -c https://github.com/AppImage/appimaged/pull/32.patch ; patch -Np1 -i 32.patch ; mkdir build ; cd build ; cmake .. -DCMAKE_INSTALL_PREFIX=/usr ; make -j9 ; cpack -V -G RPM, tried to install the RPM and I got the exact same error again.

@TheAssassin
Copy link
Member Author

@fusion809 you mean the GLib dependency error?

@fusion809
Copy link

Yep.

@fusion809
Copy link

Lol I tried to install it on Fedora Rawhide and I received a dependency error too:

$ sudo dnf install -y https://github.com/AppImage/appimaged/releases/download/continuous/appimaged-appimaged-1_alpha-gita3b100b.travis57.x86_64.rpm
Last metadata expiration check: 19:36:51 ago on Wed 21 Nov 2018 14:57:15 AEST.
appimaged-appimaged-1_alpha-gita3b100b.travis57.x86_64.rpm                           29 kB/s | 155 kB     00:05    
Error: 
 Problem: conflicting requests
  - nothing provides libarchive13 needed by appimaged-appimaged-1_alpha-gita3b100b~travis57.x86_64
  - nothing provides libglib2.0-0 needed by appimaged-appimaged-1_alpha-gita3b100b~travis57.x86_64
  - nothing provides zlib1g needed by appimaged-appimaged-1_alpha-gita3b100b~travis57.x86_64
  - nothing provides libc6 >= 2.4 needed by appimaged-appimaged-1_alpha-gita3b100b~travis57.x86_64

@TheAssassin
Copy link
Member Author

@fusion809 you should try the one provided in Artifactory. That's where Travis build artifacts are stored (except for release/continuous builds).

@fusion809
Copy link

I gave it a go earlier as well, same error, however.

@TheAssassin
Copy link
Member Author

@fusion809 if you could provide the right package names for the packages listed in that error message, I can try to overwrite them explicitly in the CPack config.

@fusion809
Copy link

Funny, RPM itself is meant to detect libraries required by files provided by the package. It does when I package RPM packages using the openSUSE Build Service. For Fedora:

  • libarchive equates to libarchive13 (provides same library libarchive.so.13).
  • glib2 equates to libglib2.0-0.
  • zlib equates to zlib1g, both provide libz.so.1.
  • glibc equates to libc6, which is provided by libc.so.6.

openSUSE:

  • libglib2.0-0 provides libraries in the packages: libgio-2_0-0, libglib-2_0-0, libgmodule-2_0-0, libgobject-2_0-0 and libgthread-2_0-0.

@TheAssassin
Copy link
Member Author

TheAssassin commented Nov 22, 2018

@fusion809 well, that may work if you build distro specific packages on the target distros. We build the RPM packages on Debian, though. Do you think the AUTOREQ stuff works in that scenario, too? I'm not sure about it.

What's funny is that AppImageLauncher's RPM packages install fine on openSUSE and Fedora (at least when I wrote the config a few months ago...).

Anyway, will put together the list for Fedora later. Thanks so far!

@fusion809
Copy link

Not sure about AUTOREQ, I use spec files for building my RPMs, not CMake.

@probonopd
Copy link
Member

@TheAssassin did you see that the RPM we are currently generating has a real strange name with the application name 2x in the filename?

@TheAssassin
Copy link
Member Author

That's something fixed in this PR, too (that MAIN_COMPONENT line).

@probonopd
Copy link
Member

Is something missing for this to be ready for merging?

@TheAssassin
Copy link
Member Author

Yes: everything. The dependency issues weren't resolved, and I didn't have any time (and neither wasn't interested enough) to look into this.

You're free to investigate this yourself.

@probonopd
Copy link
Member

Don't have the time, sorry

@probonopd
Copy link
Member

Does any RPM expert volunteer to take this over?

@azubieta
Copy link
Contributor

fixed on #91

@azubieta azubieta closed this May 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants