Skip to content

add missing <climits> include - #46

Open
peppergrayxyz wants to merge 1 commit into
linuxdeploy:masterfrom
peppergrayxyz:climits
Open

peppergrayxyz wants to merge 1 commit into
linuxdeploy:masterfrom
peppergrayxyz:climits

Conversation

@peppergrayxyz

Copy link
Copy Markdown

building on gentoo/musl fails with error:

FAILED: [code=1] src/CMakeFiles/linuxdeploy-plugin-appimage.dir/main.cpp.o
/docker-ramdisk/linuxdeploy-build-S2cK5w/linuxdeploy-plugin-appimage/src/main.cpp: In function 'std::string findAppimagetool()':
/docker-ramdisk/linuxdeploy-build-S2cK5w/linuxdeploy-plugin-appimage/src/main.cpp:21:27: error: 'PATH_MAX' was not declared in this scope
   21 |     std::vector<char> buf(PATH_MAX, '\0');
      |                           ^~~~~~~~

building on gentoo/musl fails with error:
```
FAILED: [code=1] src/CMakeFiles/linuxdeploy-plugin-appimage.dir/main.cpp.o
/docker-ramdisk/linuxdeploy-build-S2cK5w/linuxdeploy-plugin-appimage/src/main.cpp: In function 'std::string findAppimagetool()':
/docker-ramdisk/linuxdeploy-build-S2cK5w/linuxdeploy-plugin-appimage/src/main.cpp:21:27: error: 'PATH_MAX' was not declared in this scope
   21 |     std::vector<char> buf(PATH_MAX, '\0');
      |                           ^~~~~~~~
```

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
peppergrayxyz added a commit to peppergrayxyz/linuxdeploy that referenced this pull request Nov 10, 2026
The musl builds need fixes which are not merged yet:
- linuxdeploy/linuxdeploy-plugin-appimage#45
- linuxdeploy/linuxdeploy-plugin-appimage#46
- linuxdeploy/linuxdeploy-desktopfile#11
- linuxdeploy/cmake-scripts#1

Carry them in ci/upstream-patches, and apply them in build.sh and the
workflow. Revert this commit once all of them are merged.

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
peppergrayxyz added a commit to peppergrayxyz/linuxdeploy that referenced this pull request Nov 10, 2026
The musl builds need fixes which are not merged yet:
- linuxdeploy/linuxdeploy-plugin-appimage#45
- linuxdeploy/linuxdeploy-plugin-appimage#46
- linuxdeploy/linuxdeploy-desktopfile#11
- linuxdeploy/cmake-scripts#1

Carry them in ci/upstream-patches, and apply them in build.sh and the
workflow. Revert this commit once all of them are merged.

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
peppergrayxyz added a commit to peppergrayxyz/linuxdeploy that referenced this pull request Nov 10, 2026
The musl builds need fixes which are not merged yet:
- linuxdeploy/linuxdeploy-plugin-appimage#45
- linuxdeploy/linuxdeploy-plugin-appimage#46
- linuxdeploy/linuxdeploy-desktopfile#11
- linuxdeploy/cmake-scripts#1

Carry them in ci/upstream-patches, and apply them in build.sh and the
workflow. Revert this commit once all of them are merged.

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
peppergrayxyz added a commit to peppergrayxyz/linuxdeploy that referenced this pull request Nov 10, 2026
The musl builds need fixes which are not merged yet:
- linuxdeploy/linuxdeploy-plugin-appimage#45
- linuxdeploy/linuxdeploy-plugin-appimage#46
- linuxdeploy/linuxdeploy-desktopfile#11
- linuxdeploy/cmake-scripts#1

Carry them in ci/upstream-patches, and apply them in build.sh and the
workflow. Revert this commit once all of them are merged.

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
peppergrayxyz added a commit to peppergrayxyz/linuxdeploy that referenced this pull request Nov 10, 2026
The musl builds need fixes which are not merged yet:
- linuxdeploy/linuxdeploy-plugin-appimage#45
- linuxdeploy/linuxdeploy-plugin-appimage#46
- linuxdeploy/linuxdeploy-desktopfile#11
- linuxdeploy/cmake-scripts#1

Carry them in ci/upstream-patches, and apply them in build.sh and the
workflow. Revert this commit once all of them are merged.

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
peppergrayxyz added a commit to peppergrayxyz/linuxdeploy that referenced this pull request Nov 10, 2026
The musl builds need fixes which are not merged yet:
- linuxdeploy/linuxdeploy-plugin-appimage#45
- linuxdeploy/linuxdeploy-plugin-appimage#46
- linuxdeploy/linuxdeploy-desktopfile#11
- linuxdeploy/cmake-scripts#1

Carry them in ci/upstream-patches, and apply them in build.sh and the
workflow. Revert this commit once all of them are merged.

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
peppergrayxyz added a commit to peppergrayxyz/linuxdeploy that referenced this pull request Nov 10, 2026
The musl builds need fixes which are not merged yet:
- linuxdeploy/linuxdeploy-plugin-appimage#45
- linuxdeploy/linuxdeploy-plugin-appimage#46
- linuxdeploy/linuxdeploy-desktopfile#11
- linuxdeploy/cmake-scripts#1

Carry them in ci/upstream-patches, and apply them in build.sh and the
workflow. Revert this commit once all of them are merged.

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
peppergrayxyz added a commit to peppergrayxyz/linuxdeploy that referenced this pull request Nov 10, 2026
The musl builds need fixes which are not merged yet:
- linuxdeploy/linuxdeploy-plugin-appimage#45
- linuxdeploy/linuxdeploy-plugin-appimage#46
- linuxdeploy/linuxdeploy-desktopfile#11
- linuxdeploy/cmake-scripts#1

Carry them in ci/upstream-patches, and apply them in build.sh and the
workflow. Revert this commit once all of them are merged.

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
peppergrayxyz added a commit to peppergrayxyz/linuxdeploy that referenced this pull request Nov 11, 2026
The musl builds need fixes which are not merged yet:
- linuxdeploy/linuxdeploy-plugin-appimage#45
- linuxdeploy/linuxdeploy-plugin-appimage#46
- linuxdeploy/linuxdeploy-desktopfile#11
- linuxdeploy/cmake-scripts#1

Carry them in ci/upstream-patches, and apply them in build.sh and the
workflow. Revert this commit once all of them are merged.

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
peppergrayxyz added a commit to peppergrayxyz/linuxdeploy that referenced this pull request Nov 11, 2026
The musl builds need fixes which are not merged yet:
- linuxdeploy/linuxdeploy-plugin-appimage#45
- linuxdeploy/linuxdeploy-plugin-appimage#46
- linuxdeploy/linuxdeploy-desktopfile#11
- linuxdeploy/cmake-scripts#1

Carry them in ci/upstream-patches, and apply them in build.sh and the
workflow. Revert this commit once all of them are merged.

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
peppergrayxyz added a commit to peppergrayxyz/linuxdeploy that referenced this pull request Nov 11, 2026
The musl builds need fixes which are not merged yet:
- linuxdeploy/linuxdeploy-plugin-appimage#45
- linuxdeploy/linuxdeploy-plugin-appimage#46
- linuxdeploy/linuxdeploy-desktopfile#11
- linuxdeploy/cmake-scripts#1

Carry them in ci/upstream-patches, and apply them in build.sh and the
workflow. Revert this commit once all of them are merged.

Signed-off-by: Pepper Gray <hello@peppergray.xyz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant