diff options
Diffstat (limited to 'zarb-ml/mageia-dev/attachments/20120305')
10 files changed, 338 insertions, 0 deletions
diff --git a/zarb-ml/mageia-dev/attachments/20120305/631d4318/attachment-0001.obj b/zarb-ml/mageia-dev/attachments/20120305/631d4318/attachment-0001.obj new file mode 100644 index 000000000..9e8646588 --- /dev/null +++ b/zarb-ml/mageia-dev/attachments/20120305/631d4318/attachment-0001.obj @@ -0,0 +1,54 @@ +commit d66ae590be88c1e5e47088dcd250b7f9b9b4a8dc +Author: Dousig <thierry.vignaud@gmail.com> +Date: Mon Mar 5 15:31:53 2012 +0100 + + (install) fix circular reference that defeats perl's GC + + else we keep the RPM DB open several times until rpm fails: + + created transaction for installing on / (remove=0, install=0, upgrade=8) + error: rpmdb: Lock table is out of available locker entries + error: cannot open Basenames index using db4 - Cannot allocate memory (12) + error: rpmdb: Lock table is out of available locker entries + error: cannot open Group index using db4 - Cannot allocate memory (12) + error: rpmdb: Lock table is out of available locker entries + error: cannot open Requirename index using db4 - Cannot allocate memory (12) + error: rpmdb: Lock table is out of available locker entries + error: cannot open Triggername index using db4 - Cannot allocate memory (12) + error: rpmdb: Lock table is out of available locker entries + (...) + + regression introduced in commit r2946 on 2012-02-14 + (was: '(install) --test: do not display "More information on package"') + +diff --git a/NEWS b/NEWS +index 50b0dba..dfbc36d 100644 +--- a/NEWS ++++ b/NEWS +@@ -1,4 +1,5 @@ + - explicitely close the RPM DB after each transaction ++- fix circular dependency that defeats perl's GC + - warn if --allow-force, --allow-nodeps, --force or --keep is in use (mga#3127) + + Version 6.45 - 27 February 2012 +diff --git a/urpm/install.pm b/urpm/install.pm +index fc30b84..3dd9730 100644 +--- a/urpm/install.pm ++++ b/urpm/install.pm +@@ -222,6 +222,7 @@ sub install { + $urpm->{readmes} = {}; + my $index; + my $fh; ++ my $is_test = $options{test}; # fix circular reference + #- assume default value for some parameter. + $options{delta} ||= 1000; + +@@ -239,7 +240,7 @@ sub install { + my ($urpm, undef, $pkgid) = @_; + return unless defined $pkgid; + $callback_close_helper and $callback_close_helper->($db, @_); +- get_README_files($urpm, $trans, $urpm->{depslist}[$pkgid]) if !$options{test}; ++ get_README_files($urpm, $trans, $urpm->{depslist}[$pkgid]) if !$is_test; + close $fh if defined $fh; + }; + #- ensure perl does not create a circular reference below, otherwise all this won't be collected,
\ No newline at end of file diff --git a/zarb-ml/mageia-dev/attachments/20120305/631d4318/attachment.obj b/zarb-ml/mageia-dev/attachments/20120305/631d4318/attachment.obj new file mode 100644 index 000000000..9e8646588 --- /dev/null +++ b/zarb-ml/mageia-dev/attachments/20120305/631d4318/attachment.obj @@ -0,0 +1,54 @@ +commit d66ae590be88c1e5e47088dcd250b7f9b9b4a8dc +Author: Dousig <thierry.vignaud@gmail.com> +Date: Mon Mar 5 15:31:53 2012 +0100 + + (install) fix circular reference that defeats perl's GC + + else we keep the RPM DB open several times until rpm fails: + + created transaction for installing on / (remove=0, install=0, upgrade=8) + error: rpmdb: Lock table is out of available locker entries + error: cannot open Basenames index using db4 - Cannot allocate memory (12) + error: rpmdb: Lock table is out of available locker entries + error: cannot open Group index using db4 - Cannot allocate memory (12) + error: rpmdb: Lock table is out of available locker entries + error: cannot open Requirename index using db4 - Cannot allocate memory (12) + error: rpmdb: Lock table is out of available locker entries + error: cannot open Triggername index using db4 - Cannot allocate memory (12) + error: rpmdb: Lock table is out of available locker entries + (...) + + regression introduced in commit r2946 on 2012-02-14 + (was: '(install) --test: do not display "More information on package"') + +diff --git a/NEWS b/NEWS +index 50b0dba..dfbc36d 100644 +--- a/NEWS ++++ b/NEWS +@@ -1,4 +1,5 @@ + - explicitely close the RPM DB after each transaction ++- fix circular dependency that defeats perl's GC + - warn if --allow-force, --allow-nodeps, --force or --keep is in use (mga#3127) + + Version 6.45 - 27 February 2012 +diff --git a/urpm/install.pm b/urpm/install.pm +index fc30b84..3dd9730 100644 +--- a/urpm/install.pm ++++ b/urpm/install.pm +@@ -222,6 +222,7 @@ sub install { + $urpm->{readmes} = {}; + my $index; + my $fh; ++ my $is_test = $options{test}; # fix circular reference + #- assume default value for some parameter. + $options{delta} ||= 1000; + +@@ -239,7 +240,7 @@ sub install { + my ($urpm, undef, $pkgid) = @_; + return unless defined $pkgid; + $callback_close_helper and $callback_close_helper->($db, @_); +- get_README_files($urpm, $trans, $urpm->{depslist}[$pkgid]) if !$options{test}; ++ get_README_files($urpm, $trans, $urpm->{depslist}[$pkgid]) if !$is_test; + close $fh if defined $fh; + }; + #- ensure perl does not create a circular reference below, otherwise all this won't be collected,
\ No newline at end of file diff --git a/zarb-ml/mageia-dev/attachments/20120305/819e63d8/attachment-0001.html b/zarb-ml/mageia-dev/attachments/20120305/819e63d8/attachment-0001.html new file mode 100644 index 000000000..7091e45e0 --- /dev/null +++ b/zarb-ml/mageia-dev/attachments/20120305/819e63d8/attachment-0001.html @@ -0,0 +1,17 @@ +<p><br> +On Mar 5, 2012 8:03 AM, "Thierry Vignaud" <<a href="mailto:thierry.vignaud@gmail.com">thierry.vignaud@gmail.com</a>> wrote:<br> +><br> +> On 4 March 2012 10:26, D.Morgan <<a href="mailto:dmorganec@gmail.com">dmorganec@gmail.com</a>> wrote:<br> +> >> I guess he means this one,<br> +> >><br> +> >> here's Luc encountering it in BS:<br> +> >> <a href="http://www.mageia.org/pipermail/mageia-dev/2012-March/012553.html">http://www.mageia.org/pipermail/mageia-dev/2012-March/012553.html</a><br> +> >><br> +> >> and me on my cauldron box:<br> +> >> <a href="http://www.mageia.org/pipermail/mageia-dev/2012-March/012517.html">http://www.mageia.org/pipermail/mageia-dev/2012-March/012517.html</a><br> +> ><br> +> > i have this when installing buildrequires of libreoffice in the BS too<br> +><br> +> I suggest rebuilding chroot tarballs after removing /var/lib/rpm/_db*</p> +<p>I wanted to do that last night but iurt code is already removing them at the end of chroot generation<br> +</p> diff --git a/zarb-ml/mageia-dev/attachments/20120305/819e63d8/attachment.html b/zarb-ml/mageia-dev/attachments/20120305/819e63d8/attachment.html new file mode 100644 index 000000000..7091e45e0 --- /dev/null +++ b/zarb-ml/mageia-dev/attachments/20120305/819e63d8/attachment.html @@ -0,0 +1,17 @@ +<p><br> +On Mar 5, 2012 8:03 AM, "Thierry Vignaud" <<a href="mailto:thierry.vignaud@gmail.com">thierry.vignaud@gmail.com</a>> wrote:<br> +><br> +> On 4 March 2012 10:26, D.Morgan <<a href="mailto:dmorganec@gmail.com">dmorganec@gmail.com</a>> wrote:<br> +> >> I guess he means this one,<br> +> >><br> +> >> here's Luc encountering it in BS:<br> +> >> <a href="http://www.mageia.org/pipermail/mageia-dev/2012-March/012553.html">http://www.mageia.org/pipermail/mageia-dev/2012-March/012553.html</a><br> +> >><br> +> >> and me on my cauldron box:<br> +> >> <a href="http://www.mageia.org/pipermail/mageia-dev/2012-March/012517.html">http://www.mageia.org/pipermail/mageia-dev/2012-March/012517.html</a><br> +> ><br> +> > i have this when installing buildrequires of libreoffice in the BS too<br> +><br> +> I suggest rebuilding chroot tarballs after removing /var/lib/rpm/_db*</p> +<p>I wanted to do that last night but iurt code is already removing them at the end of chroot generation<br> +</p> diff --git a/zarb-ml/mageia-dev/attachments/20120305/bc89443b/attachment-0001.html b/zarb-ml/mageia-dev/attachments/20120305/bc89443b/attachment-0001.html new file mode 100644 index 000000000..66feebd0a --- /dev/null +++ b/zarb-ml/mageia-dev/attachments/20120305/bc89443b/attachment-0001.html @@ -0,0 +1,41 @@ +<p><br> +Le 5 mars 2012 10:45, "Guillaume Rousse" <<a href="mailto:guillomovitch@gmail.com">guillomovitch@gmail.com</a>> a écrit :<br> +><br> +> Le 05/03/2012 15:43, John Balcaen a écrit :<br> +>><br> +>><br> +>> Le 5 mars 2012 10:37, "Thierry Vignaud" <<a href="mailto:thierry.vignaud@gmail.com">thierry.vignaud@gmail.com</a><br> +>> <mailto:<a href="mailto:thierry.vignaud@gmail.com">thierry.vignaud@gmail.com</a>>> a écrit :<br> +>><br> +>> ><br> +>> > On 5 March 2012 15:06, Pascal Terjan <<a href="mailto:pterjan@gmail.com">pterjan@gmail.com</a><br> +>> <mailto:<a href="mailto:pterjan@gmail.com">pterjan@gmail.com</a>>> wrote:<br> +>> > >>>> urpmi fails to auto close the rpmdb, thus we keep the DB open<br> +>> more & more<br> +>> > >>>> until we break.<br> +>> > >>>><br> +>> > >>>> Just rebuild urpmi with this patch (-R is needed)<br> +>> > >>>><br> +>> > >>>> We'd this issue in the past, we fixed it but we mush have got some<br> +>> > >>>> circular dependency again somewhere.<br> +>> > >>><br> +>> > >>> Just applied and submitted to BS<br> +>> ><br> +>> > (...)<br> +>> ><br> +>> > >> the Buildrequires have been installed correctly on the BS for<br> +>> libreoffice \o/<br> +>> > ><br> +>> > > And compiz was uploaded :)<br> +>> ><br> +>> > Here's the real fix<br> +>> ><br> +>> > I'll commit it tonight and release a new urpmi<br> +>><br> +>> Is there an easy way to detect those circular dependency so we can fix<br> +>> eventually thoses ? ( at least in kde :p)<br> +><br> +> This is about circular dependencies inside perl code, what kind of help are you refering to here ?<br> +><br> +I misread the previous mail.<br> +</p> diff --git a/zarb-ml/mageia-dev/attachments/20120305/bc89443b/attachment.html b/zarb-ml/mageia-dev/attachments/20120305/bc89443b/attachment.html new file mode 100644 index 000000000..66feebd0a --- /dev/null +++ b/zarb-ml/mageia-dev/attachments/20120305/bc89443b/attachment.html @@ -0,0 +1,41 @@ +<p><br> +Le 5 mars 2012 10:45, "Guillaume Rousse" <<a href="mailto:guillomovitch@gmail.com">guillomovitch@gmail.com</a>> a écrit :<br> +><br> +> Le 05/03/2012 15:43, John Balcaen a écrit :<br> +>><br> +>><br> +>> Le 5 mars 2012 10:37, "Thierry Vignaud" <<a href="mailto:thierry.vignaud@gmail.com">thierry.vignaud@gmail.com</a><br> +>> <mailto:<a href="mailto:thierry.vignaud@gmail.com">thierry.vignaud@gmail.com</a>>> a écrit :<br> +>><br> +>> ><br> +>> > On 5 March 2012 15:06, Pascal Terjan <<a href="mailto:pterjan@gmail.com">pterjan@gmail.com</a><br> +>> <mailto:<a href="mailto:pterjan@gmail.com">pterjan@gmail.com</a>>> wrote:<br> +>> > >>>> urpmi fails to auto close the rpmdb, thus we keep the DB open<br> +>> more & more<br> +>> > >>>> until we break.<br> +>> > >>>><br> +>> > >>>> Just rebuild urpmi with this patch (-R is needed)<br> +>> > >>>><br> +>> > >>>> We'd this issue in the past, we fixed it but we mush have got some<br> +>> > >>>> circular dependency again somewhere.<br> +>> > >>><br> +>> > >>> Just applied and submitted to BS<br> +>> ><br> +>> > (...)<br> +>> ><br> +>> > >> the Buildrequires have been installed correctly on the BS for<br> +>> libreoffice \o/<br> +>> > ><br> +>> > > And compiz was uploaded :)<br> +>> ><br> +>> > Here's the real fix<br> +>> ><br> +>> > I'll commit it tonight and release a new urpmi<br> +>><br> +>> Is there an easy way to detect those circular dependency so we can fix<br> +>> eventually thoses ? ( at least in kde :p)<br> +><br> +> This is about circular dependencies inside perl code, what kind of help are you refering to here ?<br> +><br> +I misread the previous mail.<br> +</p> diff --git a/zarb-ml/mageia-dev/attachments/20120305/cffa88ee/attachment-0001.obj b/zarb-ml/mageia-dev/attachments/20120305/cffa88ee/attachment-0001.obj new file mode 100644 index 000000000..32ce67d72 --- /dev/null +++ b/zarb-ml/mageia-dev/attachments/20120305/cffa88ee/attachment-0001.obj @@ -0,0 +1,32 @@ +commit 6adc54613c28970be25150af81da0e1cce27b97f +Author: tv <tv@99302b65-d5f7-0310-b3dd-f8cd6f4e3d94> +Date: Tue Mar 24 15:39:51 2009 +0000 + + (install) revert refcount workarounds now that we fix refcounting in + r254652 ; revert both: + + - r253559 (2009-03-04) "(install) close another fd leak (needed for + drakx)" + + - and r253550 (2009-03-04: "(install) explicitely close the RPM DB on + comleting transaction (needed for drakx -- looks like refcount has + hard work)" + + + git-svn-id: svn+ssh://svn.mandriva.com/svn/soft/rpm/urpmi/trunk@254653 99302b65-d5f7-0310-b3dd-f8cd6f4e3d94 + +diff --git a/urpm/install.pm b/urpm/install.pm +index 4faab2d..a18796f 100644 +--- a/urpm/install.pm ++++ b/urpm/install.pm +@@ -289,10 +289,6 @@ sub install { + + urpm::sys::may_clean_rpmdb_shared_regions($urpm, $options{test}); + +- # explicitely close the RPM DB (needed for drakx -- looks like refcount has hard work): +- undef $db; +- undef $trans; +- + @errors; + } +
\ No newline at end of file diff --git a/zarb-ml/mageia-dev/attachments/20120305/cffa88ee/attachment.obj b/zarb-ml/mageia-dev/attachments/20120305/cffa88ee/attachment.obj new file mode 100644 index 000000000..32ce67d72 --- /dev/null +++ b/zarb-ml/mageia-dev/attachments/20120305/cffa88ee/attachment.obj @@ -0,0 +1,32 @@ +commit 6adc54613c28970be25150af81da0e1cce27b97f +Author: tv <tv@99302b65-d5f7-0310-b3dd-f8cd6f4e3d94> +Date: Tue Mar 24 15:39:51 2009 +0000 + + (install) revert refcount workarounds now that we fix refcounting in + r254652 ; revert both: + + - r253559 (2009-03-04) "(install) close another fd leak (needed for + drakx)" + + - and r253550 (2009-03-04: "(install) explicitely close the RPM DB on + comleting transaction (needed for drakx -- looks like refcount has + hard work)" + + + git-svn-id: svn+ssh://svn.mandriva.com/svn/soft/rpm/urpmi/trunk@254653 99302b65-d5f7-0310-b3dd-f8cd6f4e3d94 + +diff --git a/urpm/install.pm b/urpm/install.pm +index 4faab2d..a18796f 100644 +--- a/urpm/install.pm ++++ b/urpm/install.pm +@@ -289,10 +289,6 @@ sub install { + + urpm::sys::may_clean_rpmdb_shared_regions($urpm, $options{test}); + +- # explicitely close the RPM DB (needed for drakx -- looks like refcount has hard work): +- undef $db; +- undef $trans; +- + @errors; + } +
\ No newline at end of file diff --git a/zarb-ml/mageia-dev/attachments/20120305/e046ecb3/attachment-0001.html b/zarb-ml/mageia-dev/attachments/20120305/e046ecb3/attachment-0001.html new file mode 100644 index 000000000..c103c902d --- /dev/null +++ b/zarb-ml/mageia-dev/attachments/20120305/e046ecb3/attachment-0001.html @@ -0,0 +1,25 @@ +<p><br> +Le 5 mars 2012 10:37, "Thierry Vignaud" <<a href="mailto:thierry.vignaud@gmail.com">thierry.vignaud@gmail.com</a>> a écrit :<br> +><br> +> On 5 March 2012 15:06, Pascal Terjan <<a href="mailto:pterjan@gmail.com">pterjan@gmail.com</a>> wrote:<br> +> >>>> urpmi fails to auto close the rpmdb, thus we keep the DB open more & more<br> +> >>>> until we break.<br> +> >>>><br> +> >>>> Just rebuild urpmi with this patch (-R is needed)<br> +> >>>><br> +> >>>> We'd this issue in the past, we fixed it but we mush have got some<br> +> >>>> circular dependency again somewhere.<br> +> >>><br> +> >>> Just applied and submitted to BS<br> +><br> +> (...)<br> +><br> +> >> the Buildrequires have been installed correctly on the BS for libreoffice \o/<br> +> ><br> +> > And compiz was uploaded :)<br> +><br> +> Here's the real fix<br> +><br> +> I'll commit it tonight and release a new urpmi</p> +<p>Is there an easy way to detect those circular dependency so we can fix eventually thoses ? ( at least in kde :p)<br> +</p> diff --git a/zarb-ml/mageia-dev/attachments/20120305/e046ecb3/attachment.html b/zarb-ml/mageia-dev/attachments/20120305/e046ecb3/attachment.html new file mode 100644 index 000000000..c103c902d --- /dev/null +++ b/zarb-ml/mageia-dev/attachments/20120305/e046ecb3/attachment.html @@ -0,0 +1,25 @@ +<p><br> +Le 5 mars 2012 10:37, "Thierry Vignaud" <<a href="mailto:thierry.vignaud@gmail.com">thierry.vignaud@gmail.com</a>> a écrit :<br> +><br> +> On 5 March 2012 15:06, Pascal Terjan <<a href="mailto:pterjan@gmail.com">pterjan@gmail.com</a>> wrote:<br> +> >>>> urpmi fails to auto close the rpmdb, thus we keep the DB open more & more<br> +> >>>> until we break.<br> +> >>>><br> +> >>>> Just rebuild urpmi with this patch (-R is needed)<br> +> >>>><br> +> >>>> We'd this issue in the past, we fixed it but we mush have got some<br> +> >>>> circular dependency again somewhere.<br> +> >>><br> +> >>> Just applied and submitted to BS<br> +><br> +> (...)<br> +><br> +> >> the Buildrequires have been installed correctly on the BS for libreoffice \o/<br> +> ><br> +> > And compiz was uploaded :)<br> +><br> +> Here's the real fix<br> +><br> +> I'll commit it tonight and release a new urpmi</p> +<p>Is there an easy way to detect those circular dependency so we can fix eventually thoses ? ( at least in kde :p)<br> +</p> |