beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-30 00:31:14 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

Ports: Fix dependency install when port name is not port folder name

Browse source 
There was a bug in the way the `.port_include.sh` script handled
installing dependencies. According to the
[documentation](https://github.com/SerenityOS/serenity/tree/master/Ports#depends)
the depends array should have port names in it. The port system allows
for the name of the port to be different from the folder where port
lives. Previously the `installdepends` function would cd to the name
of the port, now it will find and run the `package.sh` that has the
line `port=$depend` in it.
This commit is contained in:
EWouters 2022-04-01 07:06:19 +02:00 committed by Linus Groh
parent 77d9a764e3
commit cc08f82ddb
Notes: sideshowbarker 2024-07-17 16:20:01 +09:00 
Author: https://github.com/EWouters
Commit: https://github.com/SerenityOS/serenity/commit/cc08f82ddb
Pull-request: https://github.com/SerenityOS/serenity/pull/13398
Reviewed-by: https://github.com/linusg
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
Ports/.port_include.sh
View file

@ -491,7 +491,7 @@ package_install_state() {
installdepends() {
for depend in "${depends[@]}"; do
if [ -z "$(package_install_state $depend)" ]; then
(cd "../$depend" && ./package.sh --auto)
(cd "$(dirname $(grep -E port=${depend} ../*/package.sh))" && ./package.sh --auto)
fi
done
}