Bug 3070 - some internal things are placed in /opt/thinlinc/bin (and sbin)
Summary: some internal things are placed in /opt/thinlinc/bin (and sbin)
Status: NEW
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Other (show other bugs)
Version: 2.1.0
Hardware: PC All
: P2 Normal
Target Milestone: MediumPrio
Assignee: Peter Åstrand
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-04-02 13:47 CEST by Pierre Ossman
Modified: 2024-06-17 12:10 CEST (History)
1 user (show)

See Also:
Acceptance Criteria:


Attachments

Description Pierre Ossman cendio 2009-04-02 13:47:41 CEST
We have a lot of junk in /opt/thinlinc/bin that are never meant to be used directly by the user. We should do a big cleanup and move things to libexec.
Comment 1 Pierre Ossman cendio 2009-04-02 14:05:13 CEST
At the same time we should also see if all .sh files really should be named that. We use .sh for files that need to be source:d, and I've noticed that a lot of them really don't need that.
Comment 2 Peter Åstrand cendio 2012-06-27 10:51:52 CEST
(In reply to comment #1)
> At the same time we should also see if all .sh files really should be named
> that. We use .sh for files that need to be source:d, and I've noticed that a
> lot of them really don't need that.

The reason for .sh is to increase performance; to avoid having to start a new shell for every little login script.
Comment 3 Pierre Ossman cendio 2022-03-16 11:28:13 CET
For reference, this is the current set of dubious commands:

bin:

hivetool
setxkbmap
tl-desktop-activate.sh
tl-desktop-builder
tl-shadow-notify
tl-wait-smartcard
vncconfig
xkbcomp

sbin:

tl-collect-licensestats
tl-limit-printers
Comment 5 Samuel Mannehed cendio 2024-05-21 15:17:23 CEST
We should consider if /opt/thinlinc/bin/tlclient-openconf should be moved as well. This is the tool used to associate ".tlclient" files with opening tlclient. Note that there might be technical reasons to why it is in path.

If possible, I think it would be logical to put it in /opt/thinlinc/lib, same place as where vncviewer is.

Note You need to log in before you can comment on or make changes to this bug.