Skip to content

Commit 7a4504e

Browse files
committed
network-manager-sstp: address review comments from Duncaen
- Remove unnecessary --prefix/--libdir - Proper indentation in configure_args - Combine .la/.a deletion in single find - Add gtk4-devel to hostmakedepends for cross-build - Remove empty lines between variables sstp-client: move dependency variables before short_desc
1 parent 7af1296 commit 7a4504e

2 files changed

Lines changed: 13 additions & 26 deletions

File tree

srcpkgs/network-manager-sstp/template

Lines changed: 7 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -3,21 +3,14 @@ pkgname=network-manager-sstp
33
version=1.3.2
44
revision=1
55
build_style=gnu-configure
6-
7-
configure_args="--prefix=/usr
8-
--libdir=/usr/lib
9-
--with-pppd-plugin-dir=/usr/lib/pppd/2.5.0
10-
--with-gnome
11-
--with-gtk4
12-
--disable-gtk-doc
13-
--disable-static"
14-
15-
hostmakedepends="pkg-config intltool autoconf automake libtool gettext-devel-tools glib-devel"
16-
6+
configure_args="--with-pppd-plugin-dir=/usr/lib/pppd/2.5.0
7+
--with-gnome
8+
--with-gtk4
9+
--disable-gtk-doc
10+
--disable-static"
11+
hostmakedepends="pkg-config intltool autoconf automake libtool gettext-devel-tools glib-devel gtk4-devel"
1712
makedepends="NetworkManager-devel glib-devel gnutls-devel gtk+3-devel gtk4-devel libnma-devel libsecret-devel ppp-devel sstp-client-devel"
18-
1913
depends="NetworkManager sstp-client"
20-
2114
short_desc="NetworkManager VPN plugin for SSTP"
2215
maintainer="kalelidev <gxiime@gmail.com>"
2316
license="GPL-2.0-or-later"
@@ -31,8 +24,7 @@ pre_configure() {
3124

3225
do_install() {
3326
make DESTDIR="${DESTDIR}" install
34-
find "${DESTDIR}" -name '*.la' -delete
35-
find "${DESTDIR}" -name '*.a' -delete
27+
find "${DESTDIR}" \( -name '*.la' -o -name '*.a' \) -delete
3628
}
3729

3830
network-manager-sstp-gnome_package() {

srcpkgs/sstp-client/template

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -3,22 +3,17 @@ pkgname=sstp-client
33
version=1.0.20
44
revision=1
55
build_style=gnu-configure
6+
configure_args="--with-runtime-dir=/var/run/sstpc
7+
--with-pppd-plugin-dir=/usr/lib/pppd/2.5.0"
8+
hostmakedepends="pkg-config libtool"
9+
makedepends="ppp-devel libevent-devel openssl-devel"
10+
depends="ppp ca-certificates"
611
short_desc="Secure Socket Tunneling Protocol (SSTP) client"
712
maintainer="kalelidev <gxiime@gmail.com>"
813
license="GPL-2.0-or-later"
914
homepage="https://gitlab.com/sstp-project/sstp-client"
1015
distfiles="https://gitlab.com/sstp-project/sstp-client/-/releases/${version}/downloads/dist-gzip/sstp-client-${version}.tar.gz"
1116
checksum="6c84b6cdcc21ebea6daeb8c5356dcdfd8681f4981a734f8485ed0b31fc30aadd"
12-
13-
_pppver=2.5.0
14-
15-
configure_args="--prefix=/usr
16-
--with-runtime-dir=/var/run/sstpc
17-
--with-pppd-plugin-dir=/usr/lib/pppd/${_pppver}"
18-
19-
hostmakedepends="pkg-config libtool"
20-
makedepends="ppp-devel libevent-devel openssl-devel"
21-
depends="ppp ca-certificates"
2217
make_dirs="/var/run/sstpc 0755 root root"
2318
# No useful testsuite upstream
2419
make_check="no"
@@ -31,6 +26,6 @@ sstp-client-devel_package() {
3126
vmove usr/lib/pkgconfig
3227
vmove usr/lib/libsstp_api.so
3328
vmove "usr/lib/*.a" 2>/dev/null || :
34-
vmove "usr/lib/pppd/${_pppver}/*.a"
29+
vmove "usr/lib/pppd/2.5.0/*.a"
3530
}
3631
}

0 commit comments

Comments
 (0)