Context Navigation

Modify ↓

#1573 closed defect (worksforme)

Space in pathname for mimeviewer not supported

Reported by:	anonymous	Owned by:	Christian Boos
Priority:	normal	Milestone:
Component:	general	Version:	0.8.1
Severity:	minor	Keywords:	mimeview highlighting
Cc:		Branch:
Release Notes:
API Changes:
Internal Changes:

Description

When using a space inside the pathname pointing to enscript, the execution of enscript fails.

[mimeviewer]
enscript_path = c:\program files\enscript\bin\enscript.exe

Debug out:

23:48:15 Trac[mimeview] WARNING: Display failed: Running (c:\program files\enscript\bin\enscript.exe --color -h -q --language=html -p - -Ejava) failed: 0, 'c:\program' is not recognized as an internal or external command,
operable program or batch file.
.

As a workaround I used the 8.3 format on windows

[mimeviewer]
enscript_path = c:\progra~1\enscript\bin\enscript.exe

Attachments (0)

Change History (7)

comment:1 by sid, 18 years ago

Keywords:	configuration added; Syntax Highlighting Enscript removed

Would quoting the enscript_path string work?

comment:2 by Christian Boos, 18 years ago

Keywords:	mimeview highlighting added; configuration removed
Milestone:	→ 0.11
Owner:	changed from Jonas Borgström to Christian Boos
Severity:	normal → minor

… or simply moving enscript elsewhere.

comment:3 by Christian Boos, 18 years ago

Resolution:	→ fixed
Status:	new → closed

Fixed in r4392.

I've not tested the enscript case (as I've wipe out enscript), but the similar change for php did work. Confirmation of the fix would be appreciated nevertheless.

I hope this doesn't break anything for Linux ;)

comment:4 by Matthew Good, 18 years ago

This should work on linux, but it prevents users from passing additional arguments to scripts, which was previously possible. Couldn't users simply add quotes around the path in trac.ini?

comment:5 by Christian Boos, 18 years ago

Resolution:	fixed
Status:	closed → reopened

Oh right, didn't think about that. Backed out r4392 in r4395.

comment:6 by Christian Boos, 18 years ago

Resolution:	→ worksforme
Status:	reopened → closed

Adding quotes works indeed perfectly.

comment:7 by Christian Boos, 18 years ago

Milestone:	0.11

Modify Ticket

Change Properties

Summary:
Description:	When using a space inside the pathname pointing to enscript, the execution of enscript fails. {{{ [mimeviewer] enscript_path = c:\program files\enscript\bin\enscript.exe }}} Debug out: {{{ 23:48:15 Trac[mimeview] WARNING: Display failed: Running (c:\program files\enscript\bin\enscript.exe --color -h -q --language=html -p - -Ejava) failed: 0, 'c:\program' is not recognized as an internal or external command, operable program or batch file. . }}} As a workaround I used the 8.3 format on windows {{{ [mimeviewer] enscript_path = c:\progra~1\enscript\bin\enscript.exe }}} You may use WikiFormatting here.
Type:		Priority:
Milestone:		Component:
Version:		Severity:
Keywords:		Cc:	Set your email in Preferences
Branch:
Release Notes:
API Changes:
Internal Changes:

Action

leave as closed The owner will remain Christian Boos.

reopen The resolution will be deleted. Next status will be 'reopened'.

change ownership to The owner will be changed from Christian Boos to the specified user.

Add Comment

Your email or username:

E-mail address and name can be saved in the Preferences .

You may use WikiFormatting here.

Attachments ↑ Description ↑

Note: See TracTickets for help on using tickets.

Download in other formats: