Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

openSeaChest .deb package misses export PATH variable #132

Open
walterav1984 opened this issue Jan 22, 2024 · 11 comments
Open

openSeaChest .deb package misses export PATH variable #132

walterav1984 opened this issue Jan 22, 2024 · 11 comments

Comments

@walterav1984
Copy link

The openSeaChest V23.12 64bit .deb package installs fine on both Ubuntu 23.04 and Debian 12.
However after install OpenSeaChest commands nor manual cannot be found nor used directly in shell from neither distro's until a specific path variable is exported or whole global path is passed:

export PATH=/usr/bin/openSeaChest:$PATH

#94 (comment)

Compared to earlier pull request the location of the binaries & manual have changed from /opt/ to /urs/bin/.

Currently openSeaChest is officially test packaged by Debian itself and uses /usr/bin/ directly without a additional subfolder openSeaChest for all its binaries.

For lowering confusion it may be better to use a /opt/ or some other local bin for this out of tree github created deb package with a correct export path?

@vonericsen
Copy link
Contributor

I will look into this.
I think this was setup prior to openSeaChest being available directly through debian's package manager.

@alexmyczko
Copy link

@walterav1984 so far nobody asked for it in Debian 12 (official backport), but it should be no problem, if someone does, and officially this should be done by BTS (debian bug tracker system), but i also accept requests on IRC (you'll figure how to find me)

@alexmyczko
Copy link

meanwhile i uploaded openseachest for debian 12, it should appear shortly at https://ftp-master.debian.org/backports-new.html

@alexmyczko
Copy link

@walterav1984 do you think it would make sense to close this issue?

@molohov
Copy link

molohov commented Sep 5, 2024

I recently upgraded from 23.03 to 24.08 and now I cannot find openSeaChest anywhere, including /usr/bin/openSeaChest. Could someone advise?

@alexmyczko
Copy link

alexmyczko commented Sep 5, 2024

@molohov what does apt policy openseachest say? and dpkg -L openseachest?

@molohov
Copy link

molohov commented Sep 5, 2024

apt policy openseachest 
openseachest:
  Installed: 24.08
  Candidate: 24.08
  Version table:
 *** 24.08 100
        100 /var/lib/dpkg/status
     23.12-1~bpo12+1 100
        100 http://deb.debian.org/debian bookworm-backports/main amd64 Packages

dpkg -L openseachest
/usr
/usr/local
/usr/local/bin
/usr/local/bin/openSeaChest
/usr/local/bin/openSeaChest/openSeaChest_Basics
/usr/local/bin/openSeaChest/openSeaChest_Configure
/usr/local/bin/openSeaChest/openSeaChest_Defect
/usr/local/bin/openSeaChest/openSeaChest_Erase
/usr/local/bin/openSeaChest/openSeaChest_Firmware
/usr/local/bin/openSeaChest/openSeaChest_Format
/usr/local/bin/openSeaChest/openSeaChest_GenericTests
/usr/local/bin/openSeaChest/openSeaChest_Info
/usr/local/bin/openSeaChest/openSeaChest_Logs
/usr/local/bin/openSeaChest/openSeaChest_NVMe
/usr/local/bin/openSeaChest/openSeaChest_PassthroughTest
/usr/local/bin/openSeaChest/openSeaChest_PowerControl
/usr/local/bin/openSeaChest/openSeaChest_Raw
/usr/local/bin/openSeaChest/openSeaChest_Reservations
/usr/local/bin/openSeaChest/openSeaChest_SMART
/usr/local/bin/openSeaChest/openSeaChest_Security
/usr/local/bin/openSeaChest/openSeaChest_ZBD
/usr/local/share
/usr/local/share/man
/usr/local/share/man/man8
/usr/local/share/man/man8/openSeaChest_Basics.8
/usr/local/share/man/man8/openSeaChest_Configure.8
/usr/local/share/man/man8/openSeaChest_Defect.8
/usr/local/share/man/man8/openSeaChest_Erase.8
/usr/local/share/man/man8/openSeaChest_Firmware.8
/usr/local/share/man/man8/openSeaChest_Format.8
/usr/local/share/man/man8/openSeaChest_GenericTests.8
/usr/local/share/man/man8/openSeaChest_Info.8
/usr/local/share/man/man8/openSeaChest_Logs.8
/usr/local/share/man/man8/openSeaChest_NVMe.8
/usr/local/share/man/man8/openSeaChest_PassthroughTest.8
/usr/local/share/man/man8/openSeaChest_PowerControl.8
/usr/local/share/man/man8/openSeaChest_Raw.8
/usr/local/share/man/man8/openSeaChest_Reservations.8
/usr/local/share/man/man8/openSeaChest_SMART.8
/usr/local/share/man/man8/openSeaChest_Security.8
/usr/local/share/man/man8/openSeaChest_ZBD.8

@vonericsen
Copy link
Contributor

I see the issue. I left it one directory too deep.
My apologies.

I have another bug reported and a feature I'm almost done with that I am also looking into so I can push another release tag with a .1 to resolve this when I have those completed. If this is needed more urgently, let me know.

vonericsen added a commit that referenced this issue Sep 5, 2024
…al/bin

This is outputting one directory too deep due to misunderstanding the format of this file.
It will now install from RPM or DEB to /usr/local/bin rather than /usr/local/bin/openSeaChest

[#132]

Signed-off-by: Tyler Erickson <[email protected]>
vonericsen added a commit that referenced this issue Sep 10, 2024
…al/bin

This is outputting one directory too deep due to misunderstanding the format of this file.
It will now install from RPM or DEB to /usr/local/bin rather than /usr/local/bin/openSeaChest

[#132]

Signed-off-by: Tyler Erickson <[email protected]>
@vonericsen
Copy link
Contributor

vonericsen commented Sep 26, 2024

I have created release v24.08.1 that fixes this in the RPM and DEB packages.
Please let me know if you have any other issues and thank you for reporting this!

@walterav1984
Copy link
Author

Dear @alexmyczko thanks for the packaging effort. If you don't mind fixing the path variables from the upstream Seagate package downstream when repackaging for Debian, this bug report may be closed.

@alexmyczko
Copy link

i guess this can be closed now

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

No branches or pull requests

4 participants