Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 738906 - dev-lang/perl: missing Prefix on Android support
Summary: dev-lang/perl: missing Prefix on Android support
Status: CONFIRMED
Alias: None
Product: Gentoo/Alt
Classification: Unclassified
Component: Prefix Support (show other bugs)
Hardware: All Other
: Normal normal (vote)
Assignee: Gentoo Prefix
URL:
Whiteboard:
Keywords: PATCH
: 818952 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-08-25 08:16 UTC by Necktwi Ozfguah
Modified: 2021-10-21 07:58 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
perl-5.30.3-gentoo-prefix.patch (perl-5.30.3-gentoo-prefix.patch,368 bytes, patch)
2020-08-25 08:16 UTC, Necktwi Ozfguah
Details | Diff
perl-5.30.3/temp/build.log (build.log,12.19 KB, text/x-log)
2020-09-04 05:23 UTC, Necktwi Ozfguah
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Necktwi Ozfguah 2020-08-25 08:16:19 UTC
Created attachment 656574 [details, diff]
perl-5.30.3-gentoo-prefix.patch

on android Gentoo-prefix, perl configure identifies build system as android and fails with following build error
https://gist.github.com/Necktwi/896df26281da60f05fa89201082af626
is the failing build.log.
I've made a patch to identify build environment as non android. find the patch as attachment to this bug report.
Comment 1 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2020-08-25 10:00:39 UTC
Which of these apply, if any? [linux-android]
./hints/linux-android.sh: line 277: getprop: command not found
 * Unable to trace static ELF: /usr/bin/ldd: ldd --version
./hints/linux.sh: line 247: 28631 Segmentation fault ${sh:-/bin/sh} -c exit

^ this doesn't look good either.
Comment 2 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2020-08-25 10:02:23 UTC
And is musl normal for android? Could be partly musl's fault
Comment 3 Necktwi Ozfguah 2020-08-26 10:08:05 UTC
(In reply to Kent Fredric (IRC: kent\n) from comment #2)
> And is musl normal for android? Could be partly musl's fault

kent\n, I've provided the patch. please check the attachment in the very first post.
Comment 4 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2020-08-26 18:02:08 UTC
(In reply to Necktwi Ozfguah from comment #3)
> (In reply to Kent Fredric (IRC: kent\n) from comment #2)
> > And is musl normal for android? Could be partly musl's fault
> 
> kent\n, I've provided the patch. please check the attachment in the very
> first post.

I need to understand the problem, and the conditions required to create the problem, not just applying patches based on "somebody submitted it".

Hence why I ask if musl is a standard part of the android-linux deployment, or if its an exotic configuration.

eg: Does this patch make sense to apply in *all* situations for android-linux, or is it only applicable on musl.

Additionally, patching configure invalidates a whole bunch of checksums, which will likely cause perl's test suite to fail.
Comment 5 Necktwi Ozfguah 2020-08-27 04:23:43 UTC
yes, the patch make sense in all situations. I've conditioned that its an android environment only if the build is made from inside the /system tree.
Comment 6 Jeroen Roovers (RETIRED) gentoo-dev 2020-08-27 15:42:48 UTC
(In reply to Necktwi Ozfguah from comment #0)
> https://gist.github.com/Necktwi/896df26281da60f05fa89201082af626
> is the failing build.log.

Please attach those build logs here.
Comment 7 Necktwi Ozfguah 2020-09-04 05:23:34 UTC
Created attachment 658232 [details]
perl-5.30.3/temp/build.log
Comment 8 Fabian Groffen gentoo-dev 2020-12-21 18:04:01 UTC
Hmmm, so this just looks like fixing a misdetection of plain Android, because Perl devs never had any sort of thing like Prefix on top of Android.

It doesn't look too bad or wrong to me...

What additional information should we provide to make this situation more understandable?
Comment 9 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2021-10-21 07:58:46 UTC
*** Bug 818952 has been marked as a duplicate of this bug. ***