#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 14 years ago by
comment:2 Changed 14 years ago by
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 14 years ago by
Owner: | set to felix winkelmann |
---|---|
Status: | new → assigned |
comment:4 Changed 14 years ago by
I checked in something in "experimental". Please give it a try and tell me whether it works for you (commit c155b54).
comment:5 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Seems to work fine. Thank you. I'm closing this ticket.
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?