#16553 closed defect (fixed)
slime: don't use cd command
Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | easye |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 1.7.0 |
Keywords: | Cc: | ||
Port: | slime |
Description
The slime port uses the cd command in the build and destroot phases, which won't work in MacPorts 1.7.0 and later, and doesn't work in MacPorts trunk today:
$ port install slime ---> Fetching slime ---> Verifying checksum(s) for slime ---> Extracting slime ---> Configuring slime ---> Building slime with target all Error: Target org.macports.build returned: invalid command name "cd" Error: Status 1 encountered during processing. $
You should remove the cd command from the portfile. See also #12914.
Attachments (1)
Change History (4)
comment:1 Changed 16 years ago by easye
Status: | new → assigned |
---|
Changed 16 years ago by ryandesign (Ryan Carsten Schmidt)
Attachment: | slime-nocd.diff added |
---|
Untested fix; before committing, confirm that the list of files installed by the port before and after applying this patch is identical
comment:2 Changed 16 years ago by easye
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Fix applied in http://trac.macports.org/changeset/39997
Did not use Ryan's suggested patch as basis of my patch (I didn't see the notification before I revisited this).
To answer my question as to what to use:
- Use inside of system invocation with conditional pipeline execution ("&&")
ex: system "cd ${worksrcpath} && DO_SOMETHING ... "
- For xinstall usage, use either the '-W' argument or use absolute paths:
ex: xinstall -W ${worksrcpath} -d ${prefix}/share/local
Note: See
TracTickets for help on using
tickets.
Hmm. What is one supposed to use instead?