starred/ProxmoxVE

Fork 0

mirror of https://github.com/community-scripts/ProxmoxVE.git synced 2026-04-25 01:06:06 +03:00

[GH-ISSUE #10797] Apache Guacamole installer is broken #2319

New issue

Closed

opened 2026-02-26 12:52:02 +03:00 by kerem · 2 comments

kerem commented

2026-02-26 12:52:02 +03:00

Owner

Originally created by @horvatbenjamin on GitHub (Jan 14, 2026).
Original GitHub issue: https://github.com/community-scripts/ProxmoxVE/issues/10797

Hi,
This commit from 2 days ago removed the "cd /root" command (github.com/community-scripts/ProxmoxVE@57c87169a9 (diff-feafcde94d)) and because of that the tar is extracted to the wrong directory (/opt/apache-guacamole/server instead of /root)
github.com/community-scripts/ProxmoxVE@dcfeaefad1/install/apache-guacamole-install.sh (L73)

I can see two possible solutions:

add cd /root before extracting the archive
add -C /root to tar command

Thanks,
Benjamin

Originally created by @horvatbenjamin on GitHub (Jan 14, 2026). Original GitHub issue: https://github.com/community-scripts/ProxmoxVE/issues/10797 Hi, This commit from 2 days ago removed the "cd /root" command (https://github.com/community-scripts/ProxmoxVE/commit/57c87169a91b2b83062c8e0cb1b8f86b90fde71f#diff-feafcde94d6b1ef66d888e2493618480b333c8952100fc2b1f735f7f430fc15dL67) and because of that the tar is extracted to the wrong directory (/opt/apache-guacamole/server instead of /root) https://github.com/community-scripts/ProxmoxVE/blob/dcfeaefad176deb447a6db3d4bb3c5e4687ec7e8/install/apache-guacamole-install.sh#L73 I can see two possible solutions: 1) add cd /root before extracting the archive 2) add -C /root to tar command Thanks, Benjamin

kerem

2026-02-26 12:52:02 +03:00

closed this issue
added the
🛑 Failure to comply with the guidelines
label

kerem commented

2026-02-26 12:52:02 +03:00

Author

Owner

@MickLesk commented on GitHub (Jan 14, 2026):

We have issue templates to use it, not to remove all. So all our actions fails and we dont link an PR to this. Otherwise, its an One-Liner, why dont you add it as PR?

@MickLesk commented on GitHub (Jan 14, 2026): We have issue templates to use it, not to remove all. So all our actions fails and we dont link an PR to this. Otherwise, its an One-Liner, why dont you add it as PR?

kerem commented

2026-02-26 12:52:02 +03:00

Author

Owner

@horvatbenjamin commented on GitHub (Jan 14, 2026):

Sorry about the template, I just created the issue from the code view and it didn't show any templates. I created a PR which adds the missing "cd /root" command before extracting the archive

@horvatbenjamin commented on GitHub (Jan 14, 2026): Sorry about the template, I just created the issue from the code view and it didn't show any templates. I created a PR which adds the missing "cd /root" command before extracting the archive

kerem referenced this issue

2026-02-26 13:34:21 +03:00

[PR #2319] [MERGED] [gh] Workflow to change date on new json files #3691

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/ProxmoxVE#2319

No description provided.

Rows
Columns