Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 337303 - Prefix/Cygwin: Invalid path "//home/" breaks Python during "make altinstall"
Summary: Prefix/Cygwin: Invalid path "//home/" breaks Python during "make altinstall"
Status: RESOLVED NEEDINFO
Alias: None
Product: Gentoo/Alt
Classification: Unclassified
Component: Prefix Support (show other bugs)
Hardware: x86 Other
: High normal (vote)
Assignee: Gentoo Prefix
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-14 16:28 UTC by Al
Modified: 2014-01-14 19:51 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
--root=/$(DESTDIR) to --root=$(DESTDIR) to avoid //home/ (python-2.6.5-r2-cygwin-prefix-1.patch,676 bytes, patch)
2010-09-14 16:33 UTC, Al
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Al 2010-09-14 16:28:09 UTC
When running "ebuild python-2.6.5-r2.ebuild install" it breaks while trying to create a directory "//home/". 

Reproducible: Always

Steps to Reproduce:
Bootstrap Prefix on Cygwin.



The reason is a parameter handling in Makefile.pre.in line 1005 and 1085:

  --root=/$(DESTDIR)

Where $DESTDIR is /home/...

I think it is an unclean approach to try to fix a defectify reltive path by prepending a slash. People should give clean pathes.

I append a little patch that works for me, if I find out how to do that. Maybe as comment in the next setp?

Al
Comment 1 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2010-09-14 16:30:38 UTC
(In reply to comment #0)

> I append a little patch that works for me, if I find out how to do that. Maybe
> as comment in the next setp?

Unified diff (diff -u <old> <new>) and there is a "Create a New Attachment" link ;)
Comment 2 Al 2010-09-14 16:33:46 UTC
Created attachment 247287 [details, diff]
--root=/$(DESTDIR)  to --root=$(DESTDIR)  to avoid //home/

This patch works for me. There is more patching required to bootstrap Python on Cygwin. This patch only addresses the given issue.
Comment 3 Fabian Groffen gentoo-dev 2014-01-14 19:51:59 UTC
this needs reevaluation