Should warn about cvs.sourceforge.net as domain in CVSROOT

Bug #772495 reported by Samuel Bronson on 2011-04-28
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Low
Unassigned

Bug Description

It seems that sourceforge now requires that CVS checkouts be from $PROJECT.cvs.sourceforge.net, not simply cvs.sourceforge.net as it was before: see http://sourceforge.net/apps/trac/sourceforge/ticket/15981, for example. However, there are still instructions out there giving CVSROOT values in the old form, like this: http://ucon64.sourceforge.net/ucon64/developers.html#3

It could save users a lot of waiting if Launchpad would warn them when the domain name part of their CVSROOT value was "cvs.sourceforge.net" (or "cvs.sf.net"), and tell them about the need to insert the SF.net project name as the first component of the domain name: as it stands, users must wait until an import is approved before receiving any indication that something is wrong, and even then they only get a rather unenlightening message such as:

2011-04-26 02:30:40 INFO [chan bzr SocketAsChannelAdapter] Opened sftp connection (server version 3)
2011-04-26 02:30:40 WARNING CVS checkout: ['cvs', '-q', '-d', ':pserver:<email address hidden>:/cvsroot/ucon64', 'co', '-Pd', 'foreign_working_tree', 'ucon64']
Traceback (most recent call last):
  File "/srv/importd.launchpad.net/production/launchpad-rev-12915/scripts/code-import-worker.py", line 82, in <module>
    sys.exit(script.main())
  File "/srv/importd.launchpad.net/production/launchpad-rev-12915/scripts/code-import-worker.py", line 77, in main
    return import_worker.run()
  File "/srv/importd.launchpad.net/production/launchpad-rev-12915/lib/lp/codehosting/codeimport/worker.py", line 452, in run
    return self._doImport()
  File "/srv/importd.launchpad.net/production/launchpad-rev-12915/lib/lp/codehosting/codeimport/worker.py", line 534, in _doImport
    foreign_tree = self.getForeignTree()
  File "/srv/importd.launchpad.net/production/launchpad-rev-12915/lib/lp/codehosting/codeimport/worker.py", line 487, in getForeignTree
    return self.foreign_tree_store.fetch(self.FOREIGN_WORKING_TREE_PATH)
  File "/srv/importd.launchpad.net/production/launchpad-rev-12915/lib/lp/codehosting/codeimport/worker.py", line 359, in fetch
    return self.fetchFromSource(target_path)
  File "/srv/importd.launchpad.net/production/launchpad-rev-12915/lib/lp/codehosting/codeimport/worker.py", line 364, in fetchFromSource
    branch.checkout()
  File "/srv/importd.launchpad.net/production/launchpad-rev-12915/lib/lp/codehosting/codeimport/foreigntree.py", line 32, in checkout
    repository.get(self.module, self.local_path)
  File "/srv/importd.launchpad.net/production/launchpad-rev-12915/lib/CVS/__init__.py", line 392, in get
    runProcessKeepAlive(args, checkout_callback, 10.0, cwd=parentdir)
  File "/srv/importd.launchpad.net/production/launchpad-rev-12915/lib/CVS/__init__.py", line 130, in runProcessKeepAlive
    raise RuntimeError('\n'.join(message))
RuntimeError: ['cvs', '-q', '-d', ':pserver:<email address hidden>:/cvsroot/ucon64', 'co', '-Pd', 'foreign_working_tree', 'ucon64'] failed (1)
>>> Error <<<
cvs [checkout aborted]: connect to cvs.sourceforge.net(216.34.181.96):2401 failed: Connection refused

Import failed:
Traceback (most recent call last):
Failure: twisted.internet.error.ProcessTerminated: A process has ended with a probable error condition: process ended with exit code 1.

<http://launchpadlibrarian.net/70417234/naesten-ucon64-trunk.log>

Curtis Hovey (sinzui) on 2011-04-28
Changed in launchpad:
status: New → Triaged
importance: Undecided → Low
tags: added: code-import feature
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers