Commit 2decfb5
committed
Resolves [FR-1434](https://lablup.atlassian.net/browse/FR-1434)
This PR addresses two important improvements to the project's Makefile:
## Changes
1. **Security Enhancement in Clean Target**:
- Fixed the `clean` target to use explicit paths instead of `cd` commands
- Prevents risk of deleting unintended directories if paths don't exist or fail
- Makes the clean process safer and more predictable
2. **Cross-Platform Compatibility**:
- Replaced `uuidgen` dependency with `openssl rand` for random ID generation
- Ensures build process works reliably in minimal Linux environments
- Maintains backward compatibility while improving portability
## Impact
These changes improve the robustness and security of the build process without breaking existing functionality.
**Checklist:**
- [x] Documentation (changes are self-documenting in Makefile)
- [x] Cross-platform compatibility verified
- [x] No breaking changes to existing build process
[FR-1434]: https://lablup.atlassian.net/browse/FR-1434?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
1 parent 9ea530b commit 2decfb5
2 files changed
+7
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
11 | | - | |
| 11 | + | |
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| |||
47 | 47 | | |
48 | 48 | | |
49 | 49 | | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
50 | 53 | | |
51 | 54 | | |
52 | 55 | | |
| |||
196 | 199 | | |
197 | 200 | | |
198 | 201 | | |
199 | | - | |
200 | | - | |
| 202 | + | |
| 203 | + | |
201 | 204 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
60 | 60 | | |
61 | 61 | | |
62 | 62 | | |
63 | | - | |
| 63 | + | |
64 | 64 | | |
65 | 65 | | |
66 | 66 | | |
| |||
0 commit comments