danfruehauf / NetworkManager-ssh

SSH VPN integration for NetworkManager
Other
253 stars 40 forks source link

Fix rpm build #84

Closed intika closed 5 years ago

intika commented 5 years ago

update & fix rpm build

danfruehauf commented 5 years ago

@intika What distribution are you building for?

The spec file in this repository is provided in a complimentary way. The proper spec files are available for fedora/centos and you can also grab rpms for those distributions.

intika commented 5 years ago

@danfruehauf iam building for opensuse/fedora/mageia ... i am playing with the project to see how i can use it from this repo... the build does not seem to be able to replace the current used plasma-nm-ssh rpm

intika commented 5 years ago

@danfruehauf now about the fix i just pr ed it to make the build command 'autoreconf -fvi && ./configure && make rpm' work otherwise it does not

danfruehauf commented 5 years ago

now about the fix i just pr ed its to make the build command 'autoreconf -fvi && ./configure && make rpm' work otherwise it does not

I'll take your word that it does not. Generally, I've been thinking of removing the spec file from this repository for while, since it can cause some grief to people who are unfamiliar with compilation and packaging. The recommended way of installing it is via yum/dnf.

intika commented 5 years ago

now about the fix i just pr ed its to make the build command 'autoreconf -fvi && ./configure && make rpm' work otherwise it does not

I'll take your word that it does not. Generally, I've been thinking of removing the spec file from this repository for while, since it can cause some grief to people who are unfamiliar with compilation and packaging. The recommended way of installing it is via yum/dnf.

Yes i know... 2 new built files where added over time since you last edited the spec file i just added them to the spec file nothing more, otherwise the compile process does not finish and return

error: Installed (but unpackaged) file(s) found:
   /usr/lib/NetworkManager/VPN/nm-ssh-service.name
   /usr/share/appdata/network-manager-ssh.metainfo.xml 
danfruehauf commented 5 years ago

I should probably also update the package for Fedora. Thanks for that!