Code review comment for lp:~widelands-dev/widelands-website/add_DISPLAY_to_update_help

Revision history for this message
kaputtnik (franku) wrote :

> Running os.path.normpath(directory) should take care of any double slashes -
> this is how I fixed up things during the last code review round. Looks like we
> have missed a few :(
 I think having paths with double slashes should be fixed where they are build. Failures like this one would otherwise be triggered again and again when one writes new code.

> Are you still having problems with the pdf?
I have managed to fixed this too by renaming "media/wlhelp/network_graphs" and run

./manage.py update_help_pdf

again. This time it was running through without problems.

So all should be fine now on the productive website. What remains is then to manage the automatic update of the help...

« Back to merge proposal