Bug 4839 - libexec/ commands are mentioned in the /opt/thinlinc/bin/ documentation
Summary: libexec/ commands are mentioned in the /opt/thinlinc/bin/ documentation
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Documentation (show other bugs)
Target Milestone: 4.15.0
Assignee: Pierre Ossman
URL:
Keywords: prosaic, wilsj_tester
Depends on:
Blocks:
 
Reported: 2013-10-10 13:27 CEST by Pierre Ossman
Modified: 2022-03-23 13:19 CET (History)
1 user (show)

See Also:
Acceptance Criteria:
* The documentation should not imply that commands that are in libexec/ should be directly executed by the user or the administrator¹ * Commands that are not meant to be directly executed by the user or the administrator should note be mentioned in the documentation² ¹ Other operations are fine, like symlinking to xstartup.d ² I.e. stuff that bug 3070 should clean up

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Pierre Ossman cendio 2013-10-10 13:27:44 CEST 
Should probably just remove them. They are referenced directly in those cases where they are to be used by the administrator.
Comment 1 Pierre Ossman cendio 2022-03-18 14:11:25 CET 
Doing this now as part of bug 7049.
Comment 4 Pierre Ossman cendio 2022-03-18 14:19:49 CET 
Should be all done now.

Not adding any release notes as this is documentation that shouldn't have been of interest for users.
Comment 5 William Sjöblom cendio 2022-03-21 17:13:18 CET 
There are still many mentions of the scripts under /opt/thinlinc/libexec/ in the TAG. This is likely a matter of further clarifying the acceptance criteria as many of these mentions make sense.

See:
> find /opt/thinlinc/libexec -type f -executable -printf "%f\n" \
>     | xargs -L1 -I% \
>         pdfgrep -n % ~/Downloads/thinlinc-4.14.0post.pdf

Let's reopen until the acceptance criteria are updated.
Comment 6 Pierre Ossman cendio 2022-03-22 09:09:53 CET 
Right. The purpose was to avoid suggesting that they should be directly used by users or administrators. Hooking them up to ThinLinc via xstartup.d is still fine.

(although ideally such things would be controlled by configuration instead)
Comment 7 William Sjöblom cendio 2022-03-23 13:19:51 CET 
• ☑ The documentation should not imply that commands that are in
  libexec/ should be directly executed by the user or the
  administrator¹
  
  No such implications are made throughout the TAG

• ☑ Commands that are not meant to be directly executed by the user or
  the administrator should not be mentioned in the documentation²
  
  All commands listed under bug 3070 have no mentions that imply
  that these should be run directly by the user. The exception being
  `hivetool', which has been broken out into bug 7873.

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