Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 63764 - _pre21 has a mistake in the syntax for emerge sync via cvs
Summary: _pre21 has a mistake in the syntax for emerge sync via cvs
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Core (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2004-09-12 04:21 UTC by Michael Cummings (RETIRED)
Modified: 2004-10-22 08:47 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Cummings (RETIRED) gentoo-dev 2004-09-12 04:21:29 UTC
_pre21, line 2377, has 
cvs -z0 -d "+cvsroot+" co gentoo-x86 -P ",portage.settings,free=1)

The -P is being interpreted by cvs (the client) as a module to check out. I believe (no expert on CVS) this directive is supposed to come earlier in the cvs call (near the -z0 -d). As is, it breaks the ability to emerge sync via cvs.
Comment 1 Michael Cummings (RETIRED) gentoo-dev 2004-09-12 16:28:11 UTC
[[[root@nomad]]] emerge sync
^[[D>>> starting initial cvs checkout with cvs://mcummings@cvs.gentoo.org:/var/cvsroot...
cvs login: can only use `login' command with the 'pserver' method
cvs [login aborted]: CVSROOT: mcummings@cvs.gentoo.org:/var/cvsroot
cvs server: cannot find module `-P' - ignored
cvs [checkout aborted]: cannot expand modules
!!! Failed to move /usr to /usr/portage
!!! [Errno 22] Invalid argument
Comment 2 Nicholas Jones (RETIRED) gentoo-dev 2004-09-12 21:10:26 UTC
Try changing it and report back to me?
Put the -P immediately after the co.
Comment 3 Michael Cummings (RETIRED) gentoo-dev 2004-09-13 04:09:52 UTC
That fixes it :)
Comment 4 Nicholas Jones (RETIRED) gentoo-dev 2004-10-22 08:47:59 UTC
Bug has been fixed and released in stable portages on or before 2.0.51-r2