On 11/01/2010 10:28 AM, Natanael Copa wrote:
> Thanks for the apkbuild.>> We are in the middle of the 2.1 release. Is the release worth delaying> for this package?>
No, this was intended for 2.2. Should I just keep it locally and wait
until 2.1 is released before sending it?
> On Mon, 2010-11-01 at 15:20 +0000, Cameron Banta wrote:>> --->> main/nginx/APKBUILD | 58 ++++++++++++++++++++++++++++++++++++++++++++>> main/nginx/nginx.confd | 7 +++++>> main/nginx/nginx.initd | 62 ++++++++++++++++++++++++++++++++++++++++++++++++>> 3 files changed, 127 insertions(+), 0 deletions(-)>> create mode 100644 main/nginx/APKBUILD>> create mode 100644 main/nginx/nginx.confd>> create mode 100644 main/nginx/nginx.initd>>>> diff --git a/main/nginx/APKBUILD b/main/nginx/APKBUILD> We normally add new packages to testing/ first>
Sorry, I can move it there.
>> new file mode 100644>> index 0000000..b35beab>> --- /dev/null>> +++ b/main/nginx/APKBUILD>> @@ -0,0 +1,58 @@>> +# Contributor: Cameron Banta<cbanta@gmail.com>>> +# Maintainer: Cameron Banta<cbanta@gmail.com>>> +pkgname=nginx>> +pkgver=0.8.53>> +pkgrel=0>> +pkgdesc="nginx [engine x] is a HTTP and reverse proxy server">> +url="nginx.org">> +license="Custom">> +depends=>> +makedepends="pcre-dev openssl-dev libcrypto1.0 zlib-dev">> +#install="$pkgname.pre-install $pkgname.post-install">> +install=>> +subpackages=>> +source="http://nginx.org/download/$pkgname-$pkgver.tar.gz>> + nginx.initd>> + nginx.confd>> + $install>> + ">> +>> +# append extra dependencies to -dev subpackage>> +# remove if not used.>> +# depends_dev="somepackage-dev"> The 3 comment lines above should be removed.>>> +>> +_builddir="$srcdir"/$pkgname-$pkgver>> +> [cutted the part that looked good]>>> --- /dev/null>> +++ b/main/nginx/nginx.initd>> @@ -0,0 +1,62 @@>> +#! /bin/sh> Should be #!/sbin/runscript>>> +>> +### BEGIN INIT INFO>> +# Provides: nginx>> +# Required-Start: $all>> +# Required-Stop: $all>> +# Default-Start: 2 3 4 5>> +# Default-Stop: 0 1 6>> +# Short-Description: starts the nginx web server>> +# Description: starts nginx using start-stop-daemon>> +### END INIT INFO> No. We use openrc (like gentoo) which does not understand the above> service dependency info.>> The init.d needs to be rewritten to fit the rest of the system.>> Thanks!>> -nc>
Ok, I'm looking for the gentoo init script.
-Cameron
---
Unsubscribe: alpine-devel+unsubscribe@lists.alpinelinux.org
Help: alpine-devel+help@lists.alpinelinux.org
---
Thanks for the apkbuild.
We are in the middle of the 2.1 release. Is the release worth delaying
for this package?
On Mon, 2010-11-01 at 15:20 +0000, Cameron Banta wrote:
> ---> main/nginx/APKBUILD | 58 ++++++++++++++++++++++++++++++++++++++++++++> main/nginx/nginx.confd | 7 +++++> main/nginx/nginx.initd | 62 ++++++++++++++++++++++++++++++++++++++++++++++++> 3 files changed, 127 insertions(+), 0 deletions(-)> create mode 100644 main/nginx/APKBUILD> create mode 100644 main/nginx/nginx.confd> create mode 100644 main/nginx/nginx.initd> > diff --git a/main/nginx/APKBUILD b/main/nginx/APKBUILD
We normally add new packages to testing/ first
> new file mode 100644> index 0000000..b35beab> --- /dev/null> +++ b/main/nginx/APKBUILD> @@ -0,0 +1,58 @@> +# Contributor: Cameron Banta <cbanta@gmail.com>> +# Maintainer: Cameron Banta <cbanta@gmail.com>> +pkgname=nginx> +pkgver=0.8.53> +pkgrel=0> +pkgdesc="nginx [engine x] is a HTTP and reverse proxy server"> +url="nginx.org"> +license="Custom"> +depends=> +makedepends="pcre-dev openssl-dev libcrypto1.0 zlib-dev"> +#install="$pkgname.pre-install $pkgname.post-install"> +install=> +subpackages=> +source="http://nginx.org/download/$pkgname-$pkgver.tar.gz> + nginx.initd> + nginx.confd> + $install> + "> +> +# append extra dependencies to -dev subpackage> +# remove if not used.> +# depends_dev="somepackage-dev"
The 3 comment lines above should be removed.
> +> +_builddir="$srcdir"/$pkgname-$pkgver> +
[cutted the part that looked good]
> --- /dev/null> +++ b/main/nginx/nginx.initd> @@ -0,0 +1,62 @@> +#! /bin/sh
Should be #!/sbin/runscript
> +> +### BEGIN INIT INFO> +# Provides: nginx> +# Required-Start: $all> +# Required-Stop: $all> +# Default-Start: 2 3 4 5> +# Default-Stop: 0 1 6> +# Short-Description: starts the nginx web server> +# Description: starts nginx using start-stop-daemon> +### END INIT INFO
No. We use openrc (like gentoo) which does not understand the above
service dependency info.
The init.d needs to be rewritten to fit the rest of the system.
Thanks!
-nc
---
Unsubscribe: alpine-devel+unsubscribe@lists.alpinelinux.org
Help: alpine-devel+help@lists.alpinelinux.org
---
On Mon, 2010-11-01 at 10:57 -0500, Cameron Banta wrote:
> On 11/01/2010 10:28 AM, Natanael Copa wrote:> > Thanks for the apkbuild.> >> > We are in the middle of the 2.1 release. Is the release worth delaying> > for this package?> >> No, this was intended for 2.2. Should I just keep it locally and wait > until 2.1 is released before sending it?
nah... It missed the 2.1 train anyways (I just tagged it and branched it
and is working on setting up the build-2-1 vserver)
Thanks!
-nc
---
Unsubscribe: alpine-devel+unsubscribe@lists.alpinelinux.org
Help: alpine-devel+help@lists.alpinelinux.org
---