generated-sources tarball for android contains a path starting with '../'
Categories
(Firefox Build System :: General, defect)
Tracking
(firefox67 fixed)
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(1 file)
The generated-sources.tar.gz artifact on android-armv7 builds includes this entry:
../android-ndk/sources/android/cpufeatures/cpu-features.c
Which means that tar
returns an error code after untarring it:
tar: Removing leading `../' from member names
tar: ../android-ndk/sources/android/cpufeatures/cpu-features.c: Member name contains '..'
tar: Exiting with failure status due to previous errors
It seems like this bogus entry should either be stripped from the tarball, or it should be relocated to a better location. The stripping could happen in package_generated_sources.py pretty easily.
This bad entry and failure to untar the file causes a searchfox run failure when I try to include android stuff.
Assignee | ||
Comment 1•6 years ago
|
||
I'm doing a searchfox run now where I ignore the tar
exit code, to see if dropping the file would cause any other problems later in the pipeline. That might help inform the correct course of action here.
Assignee | ||
Comment 2•6 years ago
|
||
Yeah just dropping the bad entry seems good enough for me.
Assignee | ||
Comment 3•6 years ago
|
||
I tried tracing the code a bit to see what was causing this entry to show up, and it looks like one of the SOURCES entries that includes this file then gets converted into GeneratedSources via code that I don't really understand. Changing this seems dangerous, so I unless somebody tells me otherwise it seems like just dropping the bad entry when building the tarball seems ok.
Assignee | ||
Comment 4•6 years ago
|
||
On Android we apparently try to include the cpufeatures.c file from the
NDK as part of the generated-sources tarball. While this works, it makes
tar drop the bogus entry for safety reasons when unpacking the archive.
It also makes tar return an error, which is undesirable for searchfox.
It's better to just skip the entry when building the tarball.
Comment 6•6 years ago
|
||
bugherder |
Description
•