Skip to content
This repository has been archived by the owner. It is now read-only.

symlinks in git repos #82

Closed
alexlep opened this issue Jun 10, 2017 · 3 comments

Comments

Projects
None yet
3 participants
@alexlep
Copy link

commented Jun 10, 2017

Once abuild is extracting zip-archive with kernel sources(fetched earlier from github to count checksums), busybox unzip is used.

			*.zip)
				msg "Unpacking $s..."
				unzip -n -q "$s" -d "$srcdir" || return 1;;

And unzip from busybox doesn't resolve symlinks (in my case this repo was used https://github.com/kashifmin/MT6582-KK-KERNEL.git). Symlinks are extracted as simple textfiles.

Unzip from busybox:

chr# ls -altrh
total 28
drwxrwxrwx    4 root     root        4.0K Jun 10 22:48 ..
-rwxr-xr-x    1 root     root          25 Jun 10 22:48 mbldenv.sh
-rwxr-xr-x    1 root     root          22 Jun 10 22:48 makeMtk
drwxrwxrwx   25 root     root        4.0K Jun 10 22:48 kernel
drwxrwxrwx    8 root     root        4.0K Jun 10 22:49 mediatek
-rwxr-xr-x    1 root     root          17 Jun 10 22:49 mk
drwxrwxrwx    4 root     root        4.0K Jun 10 22:49 .
chr# file makeMtk
makeMtk: ASCII text, with no line terminators
chr# 

Standalone unzip:

sp@sppc:~/else/test_build/MT6582-KK-KERNEL-78ea7c34b4b91dea24d8e6d0de9f1c1145981be1$ ls -altrh
total 16K
drwxrwxr-x  8 sp sp 4,0K cze 11  2014 mediatek
drwxrwxr-x 25 sp sp 4,0K cze 11  2014 kernel
drwxrwxr-x  4 sp sp 4,0K cze 11  2014 .
drwxrwxr-x  4 sp sp 4,0K cze 11 01:07 ..
lrwxrwxrwx  1 sp sp   17 cze 11 01:08 mk -> mediatek/build/mk
lrwxrwxrwx  1 sp sp   25 cze 11 01:08 mbldenv.sh -> mediatek/build/mbldenv.sh
lrwxrwxrwx  1 sp sp   22 cze 11 01:08 makeMtk -> mediatek/build/makeMtk
sp@sppc:~/else/test_build/MT6582-KK-KERNEL-78ea7c34b4b91dea24d8e6d0de9f1c1145981be1$ file makeMtk
makeMtk: symbolic link to mediatek/build/makeMtk
sp@sppc:~/else/test_build/MT6582-KK-KERNEL-78ea7c34b4b91dea24d8e6d0de9f1c1145981be1$ unzip -v
UnZip 6.00 of 20 April 2009, by Debian. Original by Info-ZIP.
...
@ollieparanoid

This comment has been minimized.

Copy link
Member

commented Jun 10, 2017

unzip is packaged in Alpine Linux. Simply add unzip to the makedepends, and it will be used instead of BusyBox' implementation. I'll add this to troubleshooting.

@MartijnBraam

This comment has been minimized.

Copy link
Member

commented Jun 10, 2017

You can also use .tar.gz sources instead of .zip sources

@alexlep alexlep closed this Jun 10, 2017

@ollieparanoid

This comment has been minimized.

Copy link
Member

commented Jul 11, 2017

Yeah, using .tar.gz is the better solution here 👍

Upstream bugreport:
https://bugs.busybox.net/show_bug.cgi?id=10031

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.