Skip to content

Commit 8c92467

Browse files
CopilotAKuHAK
andcommitted
Fix build failure: add helpful error messages for missing git submodules and update docs
Agent-Logs-Url: https://github.com/ps2homebrew/pfsshell/sessions/d8026db8-eea1-457d-ba64-95844c366c8a Co-authored-by: AKuHAK <621640+AKuHAK@users.noreply.github.com>
1 parent 59432bd commit 8c92467

5 files changed

Lines changed: 47 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,9 @@
99
"**.hpp",
1010
"**.build",
1111
"**.in",
12-
"**.yml"
12+
"**.yml",
13+
".gitmodules",
14+
"external/**"
1315
]
1416
},
1517
"pull_request": {
@@ -20,7 +22,9 @@
2022
"**.hpp",
2123
"**.build",
2224
"**.in",
23-
"**.yml"
25+
"**.yml",
26+
".gitmodules",
27+
"external/**"
2428
]
2529
}
2630
},

README.md

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -157,6 +157,26 @@ The latest Open PS2 Loader revisions have a built-in NBD server. `pfsshell/pfsfu
157157
This project can be built by using the Meson build system. For more information
158158
about the system, please visit the following location: <https://mesonbuild.com/>
159159

160+
The project uses git submodules to provide external dependencies. Before
161+
building, make sure to initialize all submodules:
162+
163+
```sh
164+
git clone --recursive https://github.com/ps2homebrew/pfsshell.git
165+
```
166+
167+
Or, if you have already cloned the repository:
168+
169+
```sh
170+
git submodule update --init --recursive
171+
```
172+
173+
Then build with Meson:
174+
175+
```sh
176+
meson setup build/
177+
meson compile -C build/
178+
```
179+
160180
## Original project
161181

162182
The original project was created by Wizard of Oz. More information about the original project can be found at the following location:

subprojects/apa/meson.build

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,12 @@
11
project('apa', 'c')
22

3+
fs = import('fs')
4+
if not fs.is_dir('include') or not fs.is_dir('src') or not fs.is_dir('irx')
5+
error('The apa include/src/irx directories are missing. ' +
6+
'This is likely because the git submodules have not been initialized. ' +
7+
'Please run:\n git submodule update --init --recursive')
8+
endif
9+
310
libfakeps2sdk_proj = subproject('fakeps2sdk')
411
libfakeps2sdk_dep = libfakeps2sdk_proj.get_variable('libfakeps2sdk_dep')
512

subprojects/iomanX/meson.build

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,12 @@
11
project('iomanX', 'c')
22

3+
fs = import('fs')
4+
if not fs.is_dir('include') or not fs.is_dir('src')
5+
error('The iomanX include/src directories are missing. ' +
6+
'This is likely because the git submodules have not been initialized. ' +
7+
'Please run:\n git submodule update --init --recursive')
8+
endif
9+
310
libfakeps2sdk_proj = subproject('fakeps2sdk')
411
libfakeps2sdk_dep = libfakeps2sdk_proj.get_variable('libfakeps2sdk_dep')
512

subprojects/pfs/meson.build

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,12 @@
11
project('pfs', 'c')
22

3+
fs = import('fs')
4+
if not fs.is_dir('include') or not fs.is_dir('src') or not fs.is_dir('irx')
5+
error('The pfs include/src/irx directories are missing. ' +
6+
'This is likely because the git submodules have not been initialized. ' +
7+
'Please run:\n git submodule update --init --recursive')
8+
endif
9+
310
libfakeps2sdk_proj = subproject('fakeps2sdk')
411
libfakeps2sdk_dep = libfakeps2sdk_proj.get_variable('libfakeps2sdk_dep')
512

0 commit comments

Comments
 (0)