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

Reply via email to