Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows vs paths vs escaping. #59

Closed
CloCkWeRX opened this issue Mar 20, 2012 · 6 comments
Closed

Windows vs paths vs escaping. #59

CloCkWeRX opened this issue Mar 20, 2012 · 6 comments
Assignees

Comments

@CloCkWeRX
Copy link

Using:
install ok: channel://pear.pdepend.org/PHP_Depend-1.0.4
install ok: channel://pear.phpunit.de/phpcpd-1.3.5
install ok: channel://pear.phpunit.de/phploc-1.6.4
install ok: channel://pear.netpirates.net/DirectoryScanner-1.1.0
install ok: channel://pear.netpirates.net/fDOMDocument-1.2.4
install ok: channel://pear.netpirates.net/fXSL-1.0.4
install ok: channel://pear.pdepend.org/staticReflection-1.0.0
install ok: channel://pear.php.net/Console_CommandLine-1.1.3
install ok: channel://pear.netpirates.net/phpDox-0.4.0
install ok: channel://pear.phpmd.org/PHP_PMD-1.3.3
install ok: channel://pear.phpunit.de/PHP_CodeBrowser-1.0.2
install ok: channel://pear.phpqatools.org/phpqatools-1.0.0

C:\Graphite>phpdox
phpdox 0.4.0 - Copyright (C) 2010 - 2012 by Arne Blankerts

[20.03.2012 - 10:40:22] Using config file 'C:\Graphite\phpdox.xml'
[20.03.2012 - 10:40:22] Registered output engine 'html'
[20.03.2012 - 10:40:22] Registered output engine 'graph'
[20.03.2012 - 10:40:22] Registered output engine 'todo'
[20.03.2012 - 10:40:22] Registered output engine 'xslrunner'
[20.03.2012 - 10:40:22] Starting to process project 'Graphite'
[20.03.2012 - 10:40:22] Starting collector

.................................................. [50]
..................................... [87]

[20.03.2012 - 10:42:07] Collector process completed
[20.03.2012 - 10:42:07] Starting generator

..

Oups... phpDox encountered a problem and has terminated!
It most likely means you've found a bug, so please file a report for this and pa
ste the stacktrace (if given) along!

Exception: TheSeer\fDOM\fDOMException
Location: C:\php\PEAR\TheSeer\fDOMDocument\fDOMDocument.php (Line 125)

loading file 'build/phpdox/raphite\Graphite\Description.php.xml' failed.

[XML-STRING] [Line: 0 - Column: 0] Warning 1549: failed to load external entity
"file:///C:/Graphite/build/phpdox/raphite/Graphite/Description.php.xml"
#0 C:\php\PEAR\TheSeer\phpDox\generator\Generator.php(153): TheSeer\phpDox\Gener

ator->processClass()
#1 C:\php\PEAR\TheSeer\phpDox\generator\Generator.php(129): TheSeer\phpDox\Gener

ator->processGlobalOnly()
#2 C:\php\PEAR\TheSeer\phpDox\Application.php(158): TheSeer\phpDox\Generator->ru

n()
#3 C:\php\PEAR\TheSeer\phpDox\CLI.php(128): TheSeer\phpDox\Application->runGener

ator()
#4 C:\php\phpdox(62): TheSeer\phpDox\CLI->run()

It looks as though the code is looking at a path containing "\G" and treating it as a special character. Recommendation: Investigate file:/// uris in the generated xml, or alternative methods to work cross platform / avoid escaping.

@ghost ghost assigned theseer Mar 25, 2012
@gureedo
Copy link

gureedo commented Oct 15, 2013

+1
even unable to checkout master branch on windows
error: unable to create file dependencies/php/classes/Gender\Gender.xml (No such file or directory)

@theseer
Copy link
Owner

theseer commented Oct 15, 2013

@gureedo: If that happens during a git checkout, this is not a bug of phpDox but rather GIT.

@theseer
Copy link
Owner

theseer commented Oct 15, 2013

@gureedo: On an unrelated note, the filename "Gender\Gender.xml" is of course sub-optimal and should get changed. That doesn't mean though it should trigger an error when checking out.

@gureedo
Copy link

gureedo commented Oct 15, 2013

In Windows file names can not contain backslash characters. This is an OS limitation.

@theseer
Copy link
Owner

theseer commented Oct 15, 2013

sigh

@theseer
Copy link
Owner

theseer commented Dec 19, 2013

All known windows related issues should be fixed with Release 0.6.2.

In case you find any windows related problems, please open a new ticket.

@theseer theseer closed this as completed Dec 19, 2013
@theseer theseer removed this from the 0.5 milestone Apr 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants