Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

sbcl: new --with-source option #48999

Closed
wants to merge 1 commit into from
Closed

sbcl: new --with-source option #48999

wants to merge 1 commit into from

Conversation

iced
Copy link

@iced iced commented Feb 9, 2016

Please note this is only part of solution, as you'll still need to add (sb-ext:set-sbcl-source-location "/usr/local/Cellar/sbcl/1.3.2") to your local sbcl config (will require bit more time than I have ATM to fix, @ weekend most likely).

@iced iced mentioned this pull request Feb 9, 2016

if build.with? "sources"
system "cp", "-R", "contrib", prefix
system "cp", "-R", "src", prefix
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The audit failure is for these lines:

        Error: 2 problems in 1 formula
sbcl:
 * Use the `cp` Ruby method instead of `system "cp"`
 * Use the `cp` Ruby method instead of `system "cp"`

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can use prefix.install %w[contrib src] instead.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ping on this comment?

@iced
Copy link
Author

iced commented Feb 9, 2016

Already commented in issue. Please fix if you got time. If not, will do it @ weekend myself.

@apjanke
Copy link
Contributor

apjanke commented Feb 24, 2016

Instead of sticking these directly under prefix, maybe it would work better going under #{pkgshare} or #{pkgshare}/#{version}. (Including the version would be more typical for other languages, but SBCL doesn't seem to version its installation, at least not within its prefix.) That way you can reference the normal /usr/local hierarchy instead of the Homebrew-specific Cellar.

@iced
Copy link
Author

iced commented Feb 25, 2016

They should be there to make other things easier. Consider them (optional) part of the LISP machine, not sources.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants