Skip to content

Portduino packaging: Move meshtastic/web out of /usr/share/doc#5323

Merged
fifieldt merged 2 commits into
meshtastic:masterfrom
vidplace7:web_path
Nov 12, 2024
Merged

Portduino packaging: Move meshtastic/web out of /usr/share/doc#5323
fifieldt merged 2 commits into
meshtastic:masterfrom
vidplace7:web_path

Conversation

@vidplace7

@vidplace7 vidplace7 commented Nov 12, 2024 •
edited
Loading

Copy link
Copy Markdown
Member

/usr/share/doc is the home of documentation on most Linux distros.
On minimal / embedded distros (such as buildroot) this directory is often removed by default to save disk space; akin to stripping manpages.

This PR moves meshtasticd/web out of /usr/share/doc and into /usr/share on Portduino platforms.

Includes a symlink from the new location to the previous location, to ease the transition.
https://manpages.debian.org/testing/debhelper/dh_link.1.en.html

@vidplace7 vidplace7 marked this pull request as draft November 12, 2024 06:58
@vidplace7 vidplace7 marked this pull request as ready for review November 12, 2024 07:10
@vidplace7 vidplace7 changed the title Portduino: Move meshtastic/web out of /usr/share/doc Portduino packaging: Move meshtastic/web out of /usr/share/doc Nov 12, 2024
fifieldt
fifieldt approved these changes Nov 12, 2024
View reviewed changes

@fifieldt fifieldt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks!

@fifieldt fifieldt merged commit 762ccdc into meshtastic:master Nov 12, 2024
@fifieldt

fifieldt commented Nov 12, 2024

Copy link
Copy Markdown
Member

@vidplace7 new error in package after merging this

sed -E s,^refs/tags/,,
+ INPUT_VERSION=2.5.12.4a11437
+ [ -z  ]
+ du --bytes --summarize --exclude=DEBIAN .debpkg/
+ awk {print $1}
+ PACKAGE_ROOT_SIZE_BYTES=69938[17](https://github.com/meshtastic/firmware/actions/runs/11793227510/job/32849279279?pr=5322#step:9:18)6
+ INPUT_INSTALLED_SIZE=68299
+ /replacetool --debian-dir:/template/DEBIAN --package:meshtasticd --version:2.5.12.4a11437 --installed-size:68299 --depends:libyaml-cpp0.7, openssl, libulfius2.7 --homepage:none --arch:amd64 --maintainer:Jonathan Bennett --description:Native Linux Meshtastic binary.
+ cp -r /template/DEBIAN .debpkg/
+ echo 2.5.12.4a11437
+ sed -E s/^v//
+ FIXED_VERSION=2.5.12.4a11437
+ readonly FIXED_VERSION
+ OWNER=--root-owner-group
+ [ false = true ]
+ readonly DEB_FILE=meshtasticd_2.5.12.4a11437_amd64.deb
+ dpkg-deb -Zgzip --root-owner-group --build .debpkg meshtasticd_2.5.12.4a11437_amd64.deb
dpkg-deb: error: maintainer script 'preinst' has bad permissions 644 (must be >=0555 and <=0775)

thebentern added a commit that referenced this pull request Nov 12, 2024
@thebentern
Revert "Portduino packaging: Move meshtastic/web out of `/usr/share/d…
7f65230 
…oc` (#5323)"

This reverts commit 762ccdc.
thebentern added a commit that referenced this pull request Nov 12, 2024
@thebentern
Revert "Portduino packaging: Move meshtastic/web out of `/usr/share/d…
a49f080 
…oc` (#5323)" (#5325)

This reverts commit 762ccdc.
@vidplace7 vidplace7 mentioned this pull request Dec 10, 2024
Merged
jeek pushed a commit to jeek/Meshtastic-Exploiteers-Hacker-Pager that referenced this pull request Jun 30, 2026
@vidplace7
Portduino packaging: Move meshtastic/web out of /usr/share/doc (mes…
0a585c5 
…htastic#5323)

* Portduino: Move meshtastic/web out of `doc`

* Remove /usr/share/doc/meshtasticd before linking
jeek pushed a commit to jeek/Meshtastic-Exploiteers-Hacker-Pager that referenced this pull request Jun 30, 2026
@thebentern
Revert "Portduino packaging: Move meshtastic/web out of `/usr/share/d…
2411330 
…oc` (meshtastic#5323)" (meshtastic#5325)

This reverts commit 0a585c5.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fifieldt fifieldt fifieldt approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vidplace7 @fifieldt