Bug 2534 - hiveconf does not support space in string lists
Summary: hiveconf does not support space in string lists
Status: NEW
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Other (show other bugs)
Version: 1.6.0
Hardware: PC Linux
: P2 Normal
Target Milestone: MediumPrio
Assignee: Pierre Ossman
URL:
Keywords: interesting_210
Depends on:
Blocks: 7149
  Show dependency treegraph
 
Reported: 2007-10-08 16:34 CEST by Peter Åstrand
Modified: 2024-02-29 16:14 CET (History)
1 user (show)

See Also:
Acceptance Criteria:


Attachments

Description Peter Åstrand cendio 2007-10-08 16:34:29 CEST
Bug discovered on OEMs system. Notes 8 installed several .desktop files with whitespace. When running tl-desktop-activate.sh, I got:

$ . tl-desktop-activate.sh
'Lotus' missing: visit the 'Notes' application group in TLDC GUI to reconfigure
'Documents.desktop' missing: visit the 'Notes' application group in TLDC GUI to reconfigure
'Lotus' missing: visit the 'Notes' application group in TLDC GUI to reconfigure
'Notes' missing: visit the 'Notes' application group in TLDC GUI to reconfigure
'8.desktop' missing: visit the 'Notes' application group in TLDC GUI to reconfigure
'Lotus' missing: visit the 'Notes' application group in TLDC GUI to reconfigure
'Presentations.desktop' missing: visit the 'Notes' application group in TLDC GUI to reconfigure
'Lotus' missing: visit the 'Notes' application group in TLDC GUI to reconfigure
'Spreadsheets.desktop' missing: visit the 'Notes' application group in TLDC GUI to reconfigure

After executing:

mv "Lotus Documents.desktop" LotusDocuments.desktop
mv "Lotus Notes 8.desktop" "LotusNotes8.desktop"
mv "Lotus Presentations.desktop" "LotusPresentations.desktop"
mv "Lotus Spreadsheets.desktop" "LotusSpreadsheets.desktop"

...things works fine.
Comment 1 Pierre Ossman cendio 2008-06-19 10:43:58 CEST
TLDC uses the API correctly. The problem is that hiveconf has no escape system for string lists, so it does not support space in the elements. We need to enhance hiveconf to solve this.
Comment 2 Peter Åstrand cendio 2010-06-02 13:02:39 CEST
Another case when this is problematic is if you want to specify a custom printer driver with RDP printer redirection. On the command line, you can do this:

tl-run-rdesktop -r "printer:io=SHARP MX-4501N PS" 

However, you cannot enter this into rdesktop_args in appservergroups.hconf, due to the spaces. 
Comment 3 Karl Mikaelsson cendio 2018-04-16 14:16:56 CEST
(In reply to comment #2)
> Another case when this is problematic is if you want to specify a custom
> printer driver with RDP printer redirection. On the command line, you can do
> this:
> 
> tl-run-rdesktop -r "printer:io=SHARP MX-4501N PS" 
> 
> However, you cannot enter this into rdesktop_args in appservergroups.hconf, due
> to the spaces. 

This specific problem feels more like bug 5721.
Comment 4 Karl Mikaelsson cendio 2018-04-16 14:17:31 CEST
This is also a problem for /vsmserver/allowed_groups. Consider "domain users@lab.lkpg.cendio.se".
Comment 6 Pierre Ossman cendio 2024-02-29 16:14:34 CET
TOML might be something to look at. It's a more formal version of the .ini style.

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