Re: [Rpm-maint] [rpm-software-management/rpm] Run binary package generation via thread pools (#226)

2017-06-13 Thread ニール・ゴンパ
This seems to work correctly on my Mac OS X 10.10 system, though admittedly I 
don't have a great test case with lots of subpackages. With ~3 subpackages 
(progs, libs, devel), it seems to work fine.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/226#issuecomment-308068138___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] fix path to external beecrypt include dir (#233)

2017-06-13 Thread ニール・ゴンパ
Conan-Kudo requested changes on this pull request.

This is on purpose. The beecrypt headers are pulled in as `#include 
` and similar. See 
[rpmio/digest_beecrypt.c](https://github.com/rpm-software-management/rpm/blob/master/rpmio/digest_beecrypt.c).

This change effectively does nothing.



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/233#pullrequestreview-43674374___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Run binary package generation via thread pools (#226)

2017-06-13 Thread Panu Matilainen
I've no prior knowledge of OMP but doesn't look half bad on first sight. 

Cosmetics aside, we can't really have half the codebase doing half-assed manual 
pthread locking here and there and another half using OMP, AIUI this is 
undefined behavior and the string pool is heavily used by non-build code as 
well.

So to set expectations straight: we'll look at merging after rpm 4.14 is 
branched off, which will be at least couple of months away still.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/226#issuecomment-308062041___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Run binary package generation via thread pools (#226)

2017-06-13 Thread Panu Matilainen
BTW it'd probably be easier and less intrusive to make buildtime and buildhost 
part of the spec struct, initialized in newSpec() or so. AFAICS all the 
relevant places are receiving spec as the argument already, with the exception 
of writeRPM().

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/226#issuecomment-308068265___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Rpm query causes corruption in the file-backed mmaped bdb regions (#232)

2017-06-13 Thread Steven Morad
We are running `berkeley-db4-4.8.30-0.3`. It may be that a later release of bdb 
will contain a fix for this, as I see that we are now 2 major versions behind 
bdb stable.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/232#issuecomment-308225271___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Rpm query causes corruption in the file-backed mmaped bdb regions (#232)

2017-06-13 Thread Jeff Johnson
@smorad: db-4.8.30 SHOULD be fine (but there really isn't any reason not to 
upgrade to more recent versions on OS X: I have direct personal/devel 
experience on OS X with all versions of BDB since db-4.5.x, been there, done 
that, WORKSFORME).

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/232#issuecomment-308305275___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint