Opened 12 years ago

Closed 12 years ago

Last modified 10 years ago

#399 closed defect (fixed)

The retrieve option for chicken-install doesn't work for local transport

Reported by: Mario Domenech Goulart Owned by: felix winkelmann
Priority: minor Milestone: 4.9.0
Component: core tools Version: 4.6.x
Keywords: chicken-install, retrieve, transport, local Cc:
Estimated difficulty:

Description

Example:

$ chicken-install -r -t local -l /home/mario/src/chicken-eggs/release/4 html-utils
retrieving ...

Warning: destination for transport `local' ignored
 html-utils located at /home/mario/src/chicken-eggs/release/4/html-utils/tags/0.6

This command doesn't actually retrieve the egg code, although

$ chicken-install -r -t svn -l https://code.call-cc.org/svn/chicken-eggs/release/4 html-utils

and

$ chicken-install -r -t http -l http://code.call-cc.org/cgi-bin/henrietta.cgi html-utils

do.

Change History (7)

comment:1 Changed 12 years ago by felix winkelmann

This is actually intentional: -r is only defined for network access, on the local file-system, this would be just a glorified cp -r. Is there a pressing reason to support this, or is the inconsistency a problem?

comment:2 Changed 12 years ago by Mario Domenech Goulart

No pressing reason, absolutely.

Let me try to explain what I'm trying to do. I want to make salmonella execute eggs tests (i.e., tests/run.scm). As far as I know, chicken-install doesn't separate the installation and testing steps. That is, if I install an egg and the installation is successful, but tests fail, I suppose chicken-install will report a failure. It would be nice if salmonella could report that the installation step is ok and one or more tests failed.

I was thinking about implementing installation/tests separation by making salmonella install eggs like it currently does, then just retrieve eggs sources, install the test-depends dependencies and run tests/run.scm if it exists (reinstalling eggs wouldn't be necessary).

The retrieve option for local transport would be handy for people who have a checkout of the eggs repo and don't want extra network traffic. salmonella currently requires a local copy of the eggs repo.

comment:3 Changed 12 years ago by felix winkelmann

Owner: set to felix winkelmann
Status: newassigned

comment:4 Changed 12 years ago by felix winkelmann

I checked in something in "experimental". Please give it a try and tell me whether it works for you (commit c155b54).

comment:5 Changed 12 years ago by Mario Domenech Goulart

Resolution: fixed
Status: assignedclosed

Seems to work fine. Thank you. I'm closing this ticket.

comment:6 Changed 12 years ago by felix winkelmann

Milestone: 4.7.04.8.0

Milestone 4.7.0 deleted

comment:7 Changed 10 years ago by felix winkelmann

Milestone: 4.8.04.9.0

Milestone 4.8.0 deleted

Note: See TracTickets for help on using tickets.