Mail archive
alpine-devel

Re: [alpine-devel] [PATCH] testing/shadowircd: new aport

From: Natanael Copa <ncopa_at_alpinelinux.org>
Date: Mon, 2 Sep 2013 10:25:45 +0200

Thanks!

a few nitpicks below.

On Mon, 2 Sep 2013 01:11:41 -0700
Sam Dodrill <shadow.h511_at_gmail.com> wrote:

> Scalable IRC daemon with many useful features
> http://www.github.com/shadowircd/shadowircd
...
> +pkgname=shadowircd
> +pkgver=6.3.3
> +pkgrel=1

initial apkbuild should have pkgrel=0

> +arch="all"
> +license="GPL2"
> +pkgusers="ircd"

whitespace errors ^^^ (whitespace at end of line)

> +depends=""
> +depends_dev="bison flex openssl-dev zlib-dev libtool"
> +makedepends="$depends_dev"

The depends_dev package holds the depends for the -dev package. I doubt
that bison, flex and libtool is needed to be able to use the
shadowirc-dev package.

...

> diff --git a/testing/shadowircd/shadowircd.pre-install b/testing/shadowircd/shadowircd.pre-install
> new file mode 100644
> index 0000000..e40f840
> --- /dev/null
> +++ b/testing/shadowircd/shadowircd.pre-install
> _at_@ -0,0 +1,3 @@
> +#!/bin/sh
> +adduser -h /var/run/shadowircd -s /bin/false -S -D ircd 2>/dev/null

we should use /sbin/nologin instead of /bin/false now as shell.

> +exit 0

I fixed the whitespace errors and applied as is otherwise.

Thanks!

-nc


---
Unsubscribe:  alpine-devel+unsubscribe_at_lists.alpinelinux.org
Help:         alpine-devel+help_at_lists.alpinelinux.org
---
Received on Mon Sep 02 2013 - 10:25:45 UTC