brp-python-hardlink fails on files that include spaces
 

Hello!

I've reported this issue with brp-python-hardlink back in January:
https://bugzilla.redhat.com/show_bug.cgi?id=783433

Could someone (a provenpackager) please look at the patch provided in
the bug report and apply it?

Thanks and best regards,
Tadej

--
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel

brp-python-hardlink fails on files that include spaces
 

On Tue, Sep 25, 2012 at 01:42:52PM +0200, Tadej Janež wrote:
> Hello!
>
> I've reported this issue with brp-python-hardlink back in January:
> https://bugzilla.redhat.com/show_bug.cgi?id=783433
>
> Could someone (a provenpackager) please look at the patch provided in
> the bug report and apply it?
>
Tangent: Are spaces in directories under site-packages legal? I can't
figure out a way to import one.

If they aren't, then I'd think that the Visualize Qt directory (and possibly
higher level directories) should be moved to %{_datadir}.

The patch you provide looks like it would still be needed as it checks all
of $RPM_BUILD_ROOT for .pyc files, not just site-packages.

-Toshio
--
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel

brp-python-hardlink fails on files that include spaces
 

On Tue, 2012-09-25 at 09:39 -0700, Toshio Kuratomi wrote:
> On Tue, Sep 25, 2012 at 01:42:52PM +0200, Tadej Janež wrote:
> > Hello!
> >
> > I've reported this issue with brp-python-hardlink back in January:
> > https://bugzilla.redhat.com/show_bug.cgi?id=783433
> >
> > Could someone (a provenpackager) please look at the patch provided in
> > the bug report and apply it?
> >
> Tangent: Are spaces in directories under site-packages legal? I can't
> figure out a way to import one.

Almost certainly a bad idea, but FWIW it is possible, using the
__import__ builtin:

# mkdir "/usr/lib64/python2.7/site-packages/this module has a space"

# echo "print('hello world')" > "/usr/lib64/python2.7/site-packages/this
module has a space/__init__.py"

# ls /usr/lib64/python2.7/site-packages/this module has a
space/__init__.py
/usr/lib64/python2.7/site-packages/this module has a space/__init__.py

# python -c "__import__('this module has a space')"
hello world

But yeah, don't do that :)

Dave

--
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel

brp-python-hardlink fails on files that include spaces
 

Toshio,

On Tue, 2012-09-25 at 09:39 -0700, Toshio Kuratomi wrote:

> The patch you provide looks like it would still be needed as it checks all
> of $RPM_BUILD_ROOT for .pyc files, not just site-packages.

thank you for reviewing the patch.

The redhat-rpm-config's upstream repository
(http://git.fedorahosted.org/cgit/redhat-rpm-config) hasn't seen any
commits since March 2010. Is it still maintained?

Maybe the easiest way to fix this is to include the patch along other
patches in
http://pkgs.fedoraproject.org/cgit/redhat-rpm-config.git/tree/ ?

Best regards,
Tadej

--
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel

brp-python-hardlink fails on files that include spaces
 

David,

On Tue, 2012-09-25 at 16:32 -0400, David Malcolm wrote:
>
> Almost certainly a bad idea, but FWIW it is possible, using the
> __import__ builtin:
>
> # mkdir "/usr/lib64/python2.7/site-packages/this module has a space"
>
> # echo "print('hello world')" > "/usr/lib64/python2.7/site-packages/this
> module has a space/__init__.py"
>
> # ls /usr/lib64/python2.7/site-packages/this module has a
> space/__init__.py
> /usr/lib64/python2.7/site-packages/this module has a space/__init__.py
>
> # python -c "__import__('this module has a space')"
> hello world

thank you for this detailed explanation.

> But yeah, don't do that :)

I'm trying to package an open source data mining program Orange
(http://orange.biolab.si/).
Apparently, they are using spaces in one of their packages:
https://bitbucket.org/biolab/orange/src/3db591b97b0c/orange/OrangeWidgets/Visualize%20Qt
I couldn't find out how they made it to work.

Best regards,
Tadej

--
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel