Skip to content
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

[Bug]: intltool: should not build on device #9360

Closed
mguinhos opened this issue Mar 11, 2022 · 4 comments
Closed

[Bug]: intltool: should not build on device #9360

mguinhos opened this issue Mar 11, 2022 · 4 comments
Labels
enhancement perl Issue is about Perl related stuff, including cpan wontfix Issue won't be fixed

Comments

@mguinhos
Copy link
Contributor

Problem description

Compilation of intltool on device is slow and expensive

What steps will reproduce the bug?

pkg install intltool -y

image

What is the expected behavior?

The package must must install without compiling on device

System information

termux-info:

Application version:
0.118.0
Packages CPU architecture:
aarch64
Subscribed repositories:
# sources.list
deb https://dl.kcubeterm.com/termux-main/ stable main
# x11-repo (sources.list.d/x11.list)
deb https://packages.termux.org/apt/termux-x11 x11 main
Updatable packages:
All packages up to date
Android version:
11
Kernel build information:
Linux localhost 4.19.87-22307827 #1 SMP PREEMPT Wed Sep 8 10:55:38 KST 2021 aarch64 Android
Device manufacturer:
samsung
Device model:
SM-N981B
@mguinhos mguinhos added the bug report Something is not working properly label Mar 11, 2022
@xtkoba xtkoba added enhancement perl Issue is about Perl related stuff, including cpan and removed bug report Something is not working properly labels Mar 11, 2022
@thunder-coding
Copy link
Member

I am against the idea to disable on-device compilation of heavy packages. On-device builds may be used to detect problems that occur only during cross compilation as well as for other variety of reasons. The most I can agree on is to issue a warning for such packages

@xtkoba
Copy link
Contributor

xtkoba commented Mar 11, 2022

@thunder-coding, that is not the point. The author means that intltool invokes cpan in the postinst script:

termux_step_create_debscripts() {
cat <<- POSTINST_EOF > ./postinst
#!$TERMUX_PREFIX/bin/bash
set -e
echo "Sideloading Perl XML::Parser..."
cpan install XML::Parser
exit 0
POSTINST_EOF
}

@Grimler91
Copy link
Member

Grimler91 commented Mar 11, 2022

This would require porting (at least) LWP::UserAgent, Encode::Locale, File::Listing, HTTP::Date, Time::Zone, HTTP::Date, HTML::Entities, HTML::Tagset, HTTP::Headers, IO::HTML, LWP::MediaTypes.

I don't think this would be worth the effort considering that:

  • intltool is more or less obsolete, in favor of gettext
  • perl modules are generally annoying to cross-compile
  • cpan install XML::Parser works without issues

intltool should depend on make and clang though, those are needed for successful installation.

@xtkoba
Copy link
Contributor

xtkoba commented Mar 11, 2022

Also should depend on libexpat.

Grimler91 added a commit that referenced this issue Mar 11, 2022
These are needed for successful compilation/installation of perl
dependencies through cpan.

See #9360.
@xtkoba xtkoba added the wontfix Issue won't be fixed label Mar 13, 2022
@xtkoba xtkoba closed this as completed Mar 13, 2022
amuramatsu pushed a commit to amuramatsu/termux-packages that referenced this issue Apr 15, 2022
These are needed for successful compilation/installation of perl
dependencies through cpan.

See termux#9360.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement perl Issue is about Perl related stuff, including cpan wontfix Issue won't be fixed
Projects
None yet
Development

No branches or pull requests

4 participants