-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
new package: sbcl #14435
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
new package: sbcl #14435
Conversation
Grimler91
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR!
ecl needs to be installed in docker image, see comment
|
|
||
| termux_step_make() { | ||
| cd $TERMUX_PKG_SRCDIR | ||
| XC_HOST="$TERMUX_PREFIX/bin/ecl --norc --c-stack 16777217" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This won't work for cross-compilation, can't run $TERMUX_PREFIX/bin/ecl in the ubuntu docker image. Instead /usr/bin/ecl from ubuntu, or some ecl compiled in termux_step_host_build, needs to be used.
Add ecl to setup-ubuntu.sh to install ubuntu's ecl in the docker image
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suppose ubuntu 22.04's ecl will be too old as well, so we might need to compile a newer ecl in a termux_step_host_build step for this to work
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we might need to compile a newer ecl in a termux_step_host_build step for this to work
In that case, it would be better to use SBCL as the host. I will try to do it!
|
|
||
| termux_step_make() { | ||
| cd $TERMUX_PKG_SRCDIR | ||
| XC_HOST="$TERMUX_PREFIX/bin/ecl --norc --c-stack 16777217" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indentation is different on this line compared to rest of the script (use tabs everywhere)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure.
| - #+haiku '("-v") | ||
| - #-haiku '("-r") | ||
| - :output stream))))))) | ||
| + (run-program "/data/data/com.termux/files/usr/bin/uname" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use @TERMUX_PREFIX@ instead of /data/data/com.termux/files/usr in patches, it is automatically sed'ed when patch is applied
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense.
|
I can't commit to this task so I'm closing this PR. I may return to it one day. |
- Fixes termux#948 - Partially fixes termux#24176 - Partially fixes termux#183 Previous attempts and progress: - termux@880983e (2016) - termux#7625 (2021) - termux#14435 (2023) This PR mainly implements the package in a termux-packages downstream-oriented way (heavy use of internal termux-packages repository code shared with other termux-packages). There is discussion regarding the upstream-oriented way of implementation from the SBCL side (trying to non-cross-compile while avoiding use of any internal termux-packages repository code) here: https://bugs.launchpad.net/sbcl/+bug/1956852
- Fixes termux#948 - Partially fixes termux#24176 - Partially fixes termux#183 Previous attempts and progress: - termux@880983e (2016) - termux#7625 (2021) - termux#14435 (2023) This PR mainly implements the package in a termux-packages downstream-oriented way (heavy use of internal termux-packages repository code shared with other termux-packages). There is discussion regarding the upstream-oriented way of implementation from the SBCL side (trying to non-cross-compile while avoiding use of any internal termux-packages repository code) here: https://bugs.launchpad.net/sbcl/+bug/1956852
- Fixes termux#948 - Partially fixes termux#24176 - Partially fixes termux#183 Previous attempts and progress: - termux@880983e (2016) - termux#7625 (2021) - termux#14435 (2023) This PR mainly implements the package in a termux-packages downstream-oriented way (heavy use of internal termux-packages repository code shared with other termux-packages). There is discussion regarding the upstream-oriented way of implementation from the SBCL side (trying to non-cross-compile while avoiding use of any internal termux-packages repository code) here: https://bugs.launchpad.net/sbcl/+bug/1956852
- Fixes #948 - Partially fixes #24176 - Partially fixes #183 Previous attempts and progress: - 880983e (2016) - #7625 (2021) - #14435 (2023) This PR mainly implements the package in a termux-packages downstream-oriented way (heavy use of internal termux-packages repository code shared with other termux-packages). There is discussion regarding the upstream-oriented way of implementation from the SBCL side (trying to non-cross-compile while avoiding use of any internal termux-packages repository code) here: https://bugs.launchpad.net/sbcl/+bug/1956852
- Fixes termux/termux-packages#948 - Partially fixes termux/termux-packages#24176 - Partially fixes termux/termux-packages#183 Previous attempts and progress: - termux/termux-packages@880983e (2016) - termux/termux-packages#7625 (2021) - termux/termux-packages#14435 (2023) This PR mainly implements the package in a termux-packages downstream-oriented way (heavy use of internal termux-packages repository code shared with other termux-packages). There is discussion regarding the upstream-oriented way of implementation from the SBCL side (trying to non-cross-compile while avoiding use of any internal termux-packages repository code) here: https://bugs.launchpad.net/sbcl/+bug/1956852
- Fixes termux#948 - Partially fixes termux#24176 - Partially fixes termux#183 Previous attempts and progress: - termux@880983e (2016) - termux#7625 (2021) - termux#14435 (2023) This PR mainly implements the package in a termux-packages downstream-oriented way (heavy use of internal termux-packages repository code shared with other termux-packages). There is discussion regarding the upstream-oriented way of implementation from the SBCL side (trying to non-cross-compile while avoiding use of any internal termux-packages repository code) here: https://bugs.launchpad.net/sbcl/+bug/1956852
SBCL needs a host compiler to be built. Since we have ECL already packaged, it is a good candidate. However, using the latest release from ECL (21.2.1) fails to build SBCL, that's why I've bumped it.
I have tested this recipe locally and it works. I need help to be able to call
eclduringtermux_step_make. Could someone help me? Thanks.