Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

#722 closed defect (fixed)

setup-helper should obey CHICKEN_REPOSITORY

Reported by: Christian Kellermann Owned by: Kon Lovett
Priority: critical Milestone:
Component: extensions Version: 4.7.x
Keywords: setup-herlpers chicken-install Cc:
Estimated difficulty:

Description (last modified by Christian Kellermann)

If one installed chicken via some package manager, say apt-get on debian and uses a CHICKEN_REPOSITORY in ones $HOME, installing something that depends on setup-helper will fail.

setup-helper will always try to install in the system wide chicken home.

Bug has been reported by Traubert on #chicken

Attachments (2)

use-install-in-repostory.patch (386 bytes) - added by Christian Kellermann 8 years ago.
obey-chicken-repository-env-var.patch (655 bytes) - added by Christian Kellermann 8 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 8 years ago by Christian Kellermann

I think setup-helper should use install-extension for installing itself.

The status quo installs itself always in (chicken-home) for whatever reason. I propose either (install-to-repository) or use the usual (install-extension) procedure provided by core.

comment:2 Changed 8 years ago by Christian Kellermann

And installation-repository-path should take CHICKEN_REPOSITORY into account if it is set.

comment:3 Changed 8 years ago by Christian Kellermann

Description: modified (diff)

Changed 8 years ago by Christian Kellermann

Changed 8 years ago by Christian Kellermann

comment:4 Changed 8 years ago by Christian Kellermann

I have attached patches for both problems

comment:5 Changed 8 years ago by felix winkelmann

Milestone: 4.8.0

comment:6 Changed 8 years ago by Kon Lovett

Status: newaccepted

Release 1.3.2 provides an unsatisfying workaround.

comment:7 Changed 8 years ago by Kon Lovett

Release 1.4.0 installs a compiled form, 'setup-helper-mod'. Slowly eggs will be converted.

comment:8 Changed 7 years ago by sjamaan

Can this ticket be closed now?

comment:9 Changed 7 years ago by Kon Lovett

Resolution: fixed
Status: acceptedclosed

comment:10 Changed 7 years ago by sjamaan

Wow that was quick! Thanks, Kon :)

Note: See TracTickets for help on using tickets.