pungi status

Mathieu Bridon bochecha at daitauha.fr
Fri Jul 10 16:21:37 UTC 2015


On Fri, 2015-07-10 at 11:05 -0500, Dennis Gilmore wrote:
> On Friday, July 10, 2015 04:58:26 PM you wrote:
> > On Fri, 2015-07-10 at 09:03 -0500, Dennis Gilmore wrote:
> > > On Friday, July 10, 2015 09:58:59 AM Mathieu Bridon wrote:
> > > > > leading up to this we hit an issue where the mounting was not
> > > > > happening correctly. it turns out that when you pass 
> > > > > os.path.join
> > > > > two
> > > > > variables the first one is silently dropped. not sure if this 
> > > > > is
> > > > > a
> > > > > bug in python or the way that os.path.join is supposed to 
> > > > > work.
> > > > 
> > > > Do you mean this?
> > > > 
> > > >     >>> os.path.join('/foo', '/bar') == '/bar'
> > > >     
> > > >     True
> > > > 
> > > > If this is what you mean, then it indeed is the expected 
> > > > behaviour:
> > > > > If a component is an absolute path, all previous components 
> > > > > are
> > > > > thrown away and joining continues from the absolute path
> > > > > component.
> > > > > 
> > > >     https://docs.python.org/2/library/os.path.html#os.path.join
> > > 
> > > This is the case, so we will have to get a patch into koji to not 
> > > us
> > > os.path.join
> > 
> > Or rather, only pass it absolute paths as the first argument.
> > 
> > What code in Koji does that?
> 
> https://git.fedorahosted.org/cgit/koji/tree/plugins/runroot.py#n54

Attached patch should fix the issue.


-- 
Mathieu
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-runroot-Avoid-squashing-path-components-with-os.path.patch
Type: text/x-patch
Size: 1179 bytes
Desc: not available
URL: <http://lists.fedoraproject.org/pipermail/rel-eng/attachments/20150710/12eb8e91/attachment.bin>


More information about the rel-eng mailing list