[Github-comments] [geany] Fix installation of the manual on Windows (#928)

Enrico Tröger notifications at xxxxx
Fri Mar 4 05:10:33 UTC 2016


I'd be fine with https://github.com/b4n/geany/commit/40a252747292780a585e32a3cc014d65895aab25. My previous attempt was motivated by making as few changes to the existing Windows layout as necessary. However, changing the doc path to be the same as on non-Windows, seems way cleaner and removes even special code. So yes, let's go for it.

Rationale behind the original reason for having "doc" on Windows on the top-level installation directory: I'd be happy if I could remember exactly :). Most probably I made this to let users find the documentation instantly by checking the installation directory.
Today I think this is not so necessary as users can easily open the documentation from within Geany using Help->Help and we also even put a clickable link to the documentation in the installation directory. These two things should be easy enough for the user, actually.

@b4n would you like to commit https://github.com/b4n/geany/commit/40a252747292780a585e32a3cc014d65895aab25 directly? I just tested it and docs are properly installed on Windows and can be opened from within Geany.
I didn't test the NSIS changes yet but will do this as my next step is creating and testing the installer.
Once the docdir change is committed in Geany, I'll update https://github.com/geany/geany-plugins/pull/376 to make this change also for G-P.

Thanks for your input and work!

---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/928#issuecomment-192107881
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160303/0a22b30a/attachment.html>


More information about the Github-comments mailing list