headers
Rawhide Report | 6 Oct 13:48
Favicon

rawhide report: 20081006 changes

New package mediawiki-imagemap
        The ImageMap extension for MediaWiki
New package php-laconica
        PHP tool for microblogging
New package vldocking
        A Java ™ docking system for JFC Swing applications
Updated Packages:

NetworkManager-pptp-0.7.0-0.11.svn4027.fc10
-------------------------------------------
* Sun Oct  5 18:00:00 2008 Lubomir Rintel <lkundrak <at> v3.sk> 1:0.7.0-11.svn4027
- Add pptp dependency (#465644)

abcm2ps-5.9.1-1.fc10
--------------------
* Sun Oct  5 18:00:00 2008 Gerard Milmeister <gemi <at> bluewin.ch> - 5.9.1-1
- new release 5.9.1

expendable-0.0.4-2.fc10
-----------------------
* Sun Oct  5 18:00:00 2008 Tim Waugh <twaugh <at> redhat.com> 0.0.4-2
- 0.0.4.  No longer need packing patch.

fakechroot-2.8-15.fc10
----------------------
* Sun Oct  5 18:00:00 2008 Axel Thimm <Axel.Thimm <at> ATrpms.net> - 2.8-15
- Fix getpeername/getsockname socklen initialization.

fakeroot-1.9.7-18.fc10
----------------------
(Continue reading)

Permalink | Reply |
headers
Ralf Corsepius | 6 Oct 14:15
Favicon

Re: rawhide report: 20081006 changes

On Mon, 2008-10-06 at 11:51 +0000, Rawhide Report wrote:

> Broken deps for ppc64
> ----------------------------------------------------------

> 	rt3-3.8.1-1.fc10.noarch requires perl(Locale::Maketext::Fuzzy)

What is going on here?

perl-Locale-Maketext-Fuzzy is a noarch package and should be available
on all architectures.

It also is present in:
http://download.fedora.redhat.com/pub/fedora/linux/development/ppc64/os/Packages/perl-Locale-Maketext-Fuzzy-0.10-2.fc9.noarch.rpm

Examining the contents also shows this package is providing what the
report is complaining above:

# rpm -q --provides -p perl-Locale-Maketext-Fuzzy-0.10-2.fc9.noarch.rpm 
warning: perl-Locale-Maketext-Fuzzy-0.10-2.fc9.noarch.rpm: Header V3 DSA
signature: NOKEY, key ID 0b86274e
perl(Locale::Maketext::Fuzzy) = 0.10
perl-Locale-Maketext-Fuzzy = 0.10-2.fc9

Ralf

--

-- 
fedora-devel-list mailing list
fedora-devel-list <at> redhat.com
https://www.redhat.com/mailman/listinfo/fedora-devel-list
(Continue reading)

Permalink | Reply |
headers
Michael Schwendt | 6 Oct 16:22

Re: rawhide report: 20081006 changes

On Mon, 06 Oct 2008 14:15:10 +0200, Ralf Corsepius wrote:

> On Mon, 2008-10-06 at 11:51 +0000, Rawhide Report wrote:
> 
> > Broken deps for ppc64
> > ----------------------------------------------------------
> 
> > 	rt3-3.8.1-1.fc10.noarch requires perl(Locale::Maketext::Fuzzy)
> 
> What is going on here?
> 
> perl-Locale-Maketext-Fuzzy is a noarch package and should be available
> on all architectures.
> 
> It also is present in:
> http://download.fedora.redhat.com/pub/fedora/linux/development/ppc64/os/Packages/perl-Locale-Maketext-Fuzzy-0.10-2.fc9.noarch.rpm

Compare with the 20081005 report, where this was not listed as a broken
dep. Package is not listed as removed and wasn't updated in this report
either. First I thought probably some other package introduced bad
Obsoletes, but "python-decoratortools" is also reported as breaking
deps although it is present:

    TurboGears-1.0.7-2.fc10.noarch requires python-decoratortools >= 0:1.4
	mock-0.9.11-1.fc10.noarch requires python-decoratortools
	python-peak-rules-0.5a1.dev-0.2569.fc10.noarch requires python-decoratortools >= 0:1.7
	python-peak-util-addons-0.6-1.fc10.noarch requires python-decoratortools >= 0:1.5
	python-peak-util-assembler-0.5-1.fc10.noarch requires python-decoratortools >= 0:1.2

Package is there:
(Continue reading)

Permalink | Reply |
headers
Jesse Keating | 6 Oct 18:20
Favicon

Re: rawhide report: 20081006 changes

On Mon, 2008-10-06 at 16:22 +0200, Michael Schwendt wrote:
> Compare with the 20081005 report, where this was not listed as a broken
> dep. Package is not listed as removed and wasn't updated in this report
> either. First I thought probably some other package introduced bad
> Obsoletes, but "python-decoratortools" is also reported as breaking
> deps although it is present:
> 
>     TurboGears-1.0.7-2.fc10.noarch requires python-decoratortools >= 0:1.4
>         mock-0.9.11-1.fc10.noarch requires python-decoratortools
>         python-peak-rules-0.5a1.dev-0.2569.fc10.noarch requires python-decoratortools >= 0:1.7
>         python-peak-util-addons-0.6-1.fc10.noarch requires python-decoratortools >= 0:1.5
>         python-peak-util-assembler-0.5-1.fc10.noarch requires python-decoratortools >= 0:1.2
> 
> Package is there:
> 
> http://download.fedora.redhat.com/pub/fedora/linux/development/ppc64/os/Packages/python-decoratortools-1.7-1.fc9.noarch.rpm

This seems to be from a bug we've been chasing in createrepo for a
while, at least in the way that we use it to make rawhide.

Error Packages/perl-Locale-Maketext-Fuzzy-0.10-2.fc9.noarch.rpm: No
Package ID found for package
perl-Locale-Maketext-Fuzzy-0.10-2.fc9.noarch, not going to add it
Error Packages/lohit-fonts-tamil-2.3.1-1.fc10.noarch.rpm: No Package ID found for package
lohit-fonts-tamil-2.3.1-1.fc10.noarch, not going to add it
Error Packages/python-decoratortools-1.7-1.fc9.noarch.rpm: No Package ID found for package
python-decoratortools-1.7-1.fc9.noarch, not going to add it

Those are the error messages we get from createrepo, so those packages
just don't make it into the repodata, which trips the spam-o-matic
(Continue reading)

Permalink | Reply |
headers
Michael Schwendt | 6 Oct 19:10

Re: rawhide report: 20081006 changes

On Mon, 06 Oct 2008 09:20:41 -0700, Jesse Keating wrote:

> This seems to be from a bug we've been chasing in createrepo for a
> while, at least in the way that we use it to make rawhide.
> 
> Error Packages/perl-Locale-Maketext-Fuzzy-0.10-2.fc9.noarch.rpm: No
> Package ID found for package
> perl-Locale-Maketext-Fuzzy-0.10-2.fc9.noarch, not going to add it
> Error Packages/lohit-fonts-tamil-2.3.1-1.fc10.noarch.rpm: No Package ID found for package
lohit-fonts-tamil-2.3.1-1.fc10.noarch, not going to add it
> Error Packages/python-decoratortools-1.7-1.fc9.noarch.rpm: No Package ID found for package
python-decoratortools-1.7-1.fc9.noarch, not going to add it
> 
> Those are the error messages we get from createrepo, so those packages
> just don't make it into the repodata, which trips the spam-o-matic
> script.  The problem is that it's not easily reproducible, it's not
> always the same packages, and it doesn't always happen.

Hmmm, an error message should be very good for somebody who is familiar
with the createrepo/Yum code:

         if po.checksum in (None, ""):
            raise MDError, "No Package ID found for package %s, not going to add it" % po

I wanted to have a brief look at where that checksum value comes from
(here with F9), but the Yum package objects use self._checksum
instead. That's weird and doesn't make sense to me yet.

--

-- 
fedora-devel-list mailing list
(Continue reading)

Permalink | Reply |
headers
Michael Schwendt | 6 Oct 19:19

Re: rawhide report: 20081006 changes

On Mon, 6 Oct 2008 19:10:01 +0200, Michael Schwendt wrote:

>          if po.checksum in (None, ""):
>             raise MDError, "No Package ID found for package %s, not going to add it" % po
>         
> I wanted to have a brief look at where that checksum value comes from
> (here with F9), but the Yum package objects use self._checksum
> instead. That's weird and doesn't make sense to me yet.

OMG, it's using properties further down in the same file. =:-O
So much about readability...

--

-- 
fedora-devel-list mailing list
fedora-devel-list <at> redhat.com
https://www.redhat.com/mailman/listinfo/fedora-devel-list
Permalink | Reply |
headers
Jesse Keating | 6 Oct 19:19
Favicon

Re: rawhide report: 20081006 changes

On Mon, 2008-10-06 at 19:10 +0200, Michael Schwendt wrote:
> 
> Hmmm, an error message should be very good for somebody who is
> familiar
> with the createrepo/Yum code:
> 
>          if po.checksum in (None, ""):
>             raise MDError, "No Package ID found for package %s, not
> going to add it" % po
>         
> 
> I wanted to have a brief look at where that checksum value comes from
> (here with F9), but the Yum package objects use self._checksum
> instead. That's weird and doesn't make sense to me yet.

The yum folks are still looking into it.  The really bizarre thing is
that there is no problem with the package itself, in fact it's seen in
other arches, and in those other arch directories it's a hardlink back
to the actual file somewhere else on the filesystem.  That's the way it
is for each of those arch trees.  It's a real headscratcher of a
problem.

-- 
Jesse Keating
Fedora -- Freedom² is a feature!
identi.ca: http://identi.ca/jkeating

--

-- 
fedora-devel-list mailing list
(Continue reading)

Permalink | Reply |
headers
Michael Schwendt | 6 Oct 20:33

Re: rawhide report: 20081006 changes

On Mon, 06 Oct 2008 10:19:46 -0700, Jesse Keating wrote:

> The yum folks are still looking into it.  The really bizarre thing is
> that there is no problem with the package itself, in fact it's seen in
> other arches, and in those other arch directories it's a hardlink back
> to the actual file somewhere else on the filesystem.  That's the way it
> is for each of those arch trees.  It's a real headscratcher of a
> problem.

Unless I'm misreading the code, it's a race-condition. mash creates the
metadata for all architectures in parallel. One process per arch. The
createrepo checksum cache file I/O [1] is not protected against concurrent
access. With odd timing, it can happen that one process reads an empty
checksum file that has just been opened for writing by another process.
Give it a try. Add a safety check after reading a checksum file.

--

-- 
fedora-devel-list mailing list
fedora-devel-list <at> redhat.com
https://www.redhat.com/mailman/listinfo/fedora-devel-list
Permalink | Reply |
headers
Jesse Keating | 6 Oct 20:41
Favicon

Re: rawhide report: 20081006 changes

On Mon, 2008-10-06 at 20:33 +0200, Michael Schwendt wrote:
> On Mon, 06 Oct 2008 10:19:46 -0700, Jesse Keating wrote:
> 
> > The yum folks are still looking into it.  The really bizarre thing is
> > that there is no problem with the package itself, in fact it's seen in
> > other arches, and in those other arch directories it's a hardlink back
> > to the actual file somewhere else on the filesystem.  That's the way it
> > is for each of those arch trees.  It's a real headscratcher of a
> > problem.
> 
> Unless I'm misreading the code, it's a race-condition. mash creates the
> metadata for all architectures in parallel. One process per arch. The
> createrepo checksum cache file I/O [1] is not protected against concurrent
> access. With odd timing, it can happen that one process reads an empty
> checksum file that has just been opened for writing by another process.
> Give it a try. Add a safety check after reading a checksum file.
> 

That tickles a part of my brain, I think I speculated that this was the
problem while in the shower or something, and then completely forgot to
pursue it once I got ready to work.  It is a possibility.

-- 
Jesse Keating
Fedora -- Freedom² is a feature!
identi.ca: http://identi.ca/jkeating

--

-- 
fedora-devel-list mailing list
(Continue reading)

Permalink | Reply |
headers
James Antill | 8 Oct 18:44
Favicon

Re: rawhide report: 20081006 changes

On Mon, 2008-10-06 at 11:41 -0700, Jesse Keating wrote:
> On Mon, 2008-10-06 at 20:33 +0200, Michael Schwendt wrote:
> > On Mon, 06 Oct 2008 10:19:46 -0700, Jesse Keating wrote:
> > 
> > > The yum folks are still looking into it.  The really bizarre thing is
> > > that there is no problem with the package itself, in fact it's seen in
> > > other arches, and in those other arch directories it's a hardlink back
> > > to the actual file somewhere else on the filesystem.  That's the way it
> > > is for each of those arch trees.  It's a real headscratcher of a
> > > problem.
> > 
> > Unless I'm misreading the code, it's a race-condition. mash creates the
> > metadata for all architectures in parallel. One process per arch. The
> > createrepo checksum cache file I/O [1] is not protected against concurrent
> > access. With odd timing, it can happen that one process reads an empty
> > checksum file that has just been opened for writing by another process.
> > Give it a try. Add a safety check after reading a checksum file.
> > 
> 
> That tickles a part of my brain, I think I speculated that this was the
> problem while in the shower or something, and then completely forgot to
> pursue it once I got ready to work.  It is a possibility.

 This should fix it:

http://koji.fedoraproject.org/koji/taskinfo?taskID=868762

...the patch being basically:

-            csumo = open(csumfile, 'w')
(Continue reading)

Permalink | Reply |
headers
Michael Schwendt | 8 Oct 20:31

Re: rawhide report: 20081006 changes

On Wed, 08 Oct 2008 12:44:35 -0400, James Antill wrote:

[race in mash+createrepo]

> This should fix it:
> 
> http://koji.fedoraproject.org/koji/taskinfo?taskID=868762
> 
> ...the patch being basically:
> 
> -            csumo = open(csumfile, 'w')
> +
> +            #  This is atomic cache creation via. rename, so we can have two
> +            # tasks using the same cachedir ... mash does this.
> +            csumo = tempfile.NamedTemporaryFile(mode='w', dir=self.crp_cachedir
>              csumo.write(checksum)
>              csumo.close()
> +            os.rename(csumo.name, csumfile)
> 
> 
> ...it'll get upstream as soon as I can push :).

Looks good. It's the most you can do in createrepo without adding locks.
A small penalty remains, because the same checksum may be generated more
than once in a race-condition. Neglectable IMO.

--

-- 
fedora-devel-list mailing list
fedora-devel-list <at> redhat.com
https://www.redhat.com/mailman/listinfo/fedora-devel-list
(Continue reading)

Permalink | Reply |
headers
Alex Lancaster | 7 Oct 06:33

Re: rawhide report: 20081006 changes

[...]

Quick update on broken deps investigation with latest rawhide:

> Broken deps for i386
> ----------------------------------------------------------
> 	asterisk-mobile-1.6.0-0.21.beta9.fc10.i386 requires libbluetooth.so.2

Already a response on:

https://bugzilla.redhat.com/show_bug.cgi?id=465661

looks like some new packages need a review before asterisk 1.6.0 will
build properly.

> 	gnuradio-3.1.2-2.fc9.i386 requires libcppunit-1.12.so.0

I took the liberty of rebuilding this package for broken deps because
of libcppunit soname bump:

https://koji.fedoraproject.org/koji/buildinfo?buildID=52252

it had been rebuilt for F-10 before, but for some reason the build was
cancelled:

https://koji.fedoraproject.org/koji/buildinfo?buildID=52252

> 	gtkmozembedmm-1.4.2.cvs20060817-21.fc10.i386 requires gecko-libs = 0:1.9.0.1

Opened up a new bug requesting clarification:
(Continue reading)

Permalink | Reply |
headers
Patrice Dumas | 7 Oct 09:27
Favicon

Re: rawhide report: 20081006 changes

On Mon, Oct 06, 2008 at 09:33:29PM -0700, Alex Lancaster wrote:
> 
> > 	gnuradio-3.1.2-2.fc9.i386 requires libcppunit-1.12.so.0
> 
> I took the liberty of rebuilding this package for broken deps because
> of libcppunit soname bump:
> 
> https://koji.fedoraproject.org/koji/buildinfo?buildID=52252

Umh, I didn't announced cppunit soname bump because I thought that
nothing ended up being linked at runtime with cppunit... Looks like I
was wrong:
# repoquery  --whatrequires libcppunit-1.12.so.0
syncevolution-0:0.7-3.fc10.i386
gnuradio-0:3.1.2-2.fc9.i386

However these may be spurious deps, I'll have a look.

--
Pat

--

-- 
fedora-devel-list mailing list
fedora-devel-list <at> redhat.com
https://www.redhat.com/mailman/listinfo/fedora-devel-list
Permalink | Reply |

Gmane