Content-Type: text/plain; charset=utf-8
Precedence: list
Mime-Version: 1.0 (Mac OS X Mail 9.2 \(3112\))
Subject: Re: [alpine-aports] [PATCH] testing/mongodb: fix for #5117
From: Marc Vertes <marc.vertes@ugrid.net>
In-Reply-To: <6B28E7B4-454F-4391-AA4E-498D73DD811A@ugrid.net>
Date: Tue, 23 Feb 2016 18:26:12 +0100
Cc: alpine-aports@lists.alpinelinux.org
Content-Transfer-Encoding: quoted-printable
Message-Id: <C0783129-1A80-4D3B-8D2F-6D76A733095E@ugrid.net>
References: <1456247469-958-1-git-send-email-marc.vertes@ugrid.net> <20160223191612.6f25f25b@vostro> <6B28E7B4-454F-4391-AA4E-498D73DD811A@ugrid.net>
To: Timo Teras <timo.teras@iki.fi>


> Le 23 f=C3=A9vr. 2016 =C3=A0 18:18, Marc Vertes =
<marc.vertes@ugrid.net> a =C3=A9crit :
>=20
>=20
>>=20
>> Technically the getrlimit() is perfectly ok to do on all platforms. =
But
>> we know glibc returns rlimit anway, so it can be skipped there.
>>=20
>> I would have put all of the base/size twiddling inside the if(). Then
>> struct rlimit does not need initialization either. But thats a minor
>> thingy. You care to fix that, or should I just apply this?
>>=20
> Ok, let met fix, test and resubmit.

Adjusting the base/size outside of if() is necessary if init is not
performed by main thread, which is the case for mongo shell. If you
agree with that, you can commit as it is.

Thanks
--
Marc=


---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---