这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@aadcg
Copy link

@aadcg aadcg commented Jan 10, 2023

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 ecl during termux_step_make. Could someone help me? Thanks.

Copy link
Member

@Grimler91 Grimler91 left a 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"
Copy link
Member

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

Copy link
Member

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

Copy link
Author

@aadcg aadcg Jan 10, 2023

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"
Copy link
Member

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)

Copy link
Author

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"
Copy link
Member

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

Copy link
Author

Choose a reason for hiding this comment

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

Makes sense.

@aadcg
Copy link
Author

aadcg commented Jul 25, 2023

I can't commit to this task so I'm closing this PR. I may return to it one day.

@aadcg aadcg closed this Jul 25, 2023
robertkirkman added a commit to robertkirkman/termux-packages that referenced this pull request Apr 18, 2025
- 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
robertkirkman added a commit to robertkirkman/termux-packages that referenced this pull request Apr 18, 2025
- 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
robertkirkman added a commit to robertkirkman/termux-packages that referenced this pull request Apr 18, 2025
- 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
robertkirkman added a commit that referenced this pull request Apr 26, 2025
- 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
termux-pacman-bot added a commit to termux-pacman/termux-packages that referenced this pull request Apr 26, 2025
- 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
shadichy pushed a commit to shadichy/termux-packages that referenced this pull request Apr 28, 2025
- 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants