Re: [Fedora-r-devel-list] R2rpm call for tester

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday 06 April 2010 21:48:36 Pierre-Yves wrote:
> >
> > Nice work, I am testing it now and it has really improved a lot. :-)
> >
> > I use the attached patch to get the %{name} translation right.
> 
> Thanks, I will look into this :)

FWIW it would be nice to have the packname (or the spec name for that matter) 
earlier, as soon as possible. If we had that all those extra lines would go 
away. I tried to play a bit with it but failed.

Then the code could be reduced to:

    dirname = Popen(["rpm", "-E", '%' + tag], stdout=PIPE).stdout.read()[:-1]
    return dirname.replace("%{name}", self.packname).replace("%name", 
self.packname)

(Yes, I am abusing code brevity, but you get an idea). :-)
-- 
José Abílio
_______________________________________________
r-devel mailing list
r-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/r-devel


[Index of Archives]     [Fedora Users]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux