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 tree / graph
 
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
Add an attachment (proposed patch, testcase, etc.)

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.