#570: Remote Debugger - enhancements to integrate
--------------------------+-------------------
Reporter: rchyla | Owner: simko
Type: enhancement | Status: new
Priority: major | Milestone:
Component: *general* | Version:
Resolution: | Keywords:
--------------------------+-------------------
Comment (by simko):
Instead of creating a new subdirectory in `modules`, please move the
`remote_debugger` files into `miscutil`, where smaller utilities like
that usually live. I planned to move the files during merging when I
looked at this branch several months ago, but I have not got to it
yet, so it would be great if you can do it yourself.
Note that in its current state, the branch does not work, since the
file `webinterface_handler_wsgi` contains unresolved merge conflicts,
see `git grep '>>>>>'`. So this is to be amended.
While doing so, please squash everything together in order to avoid
unnecessary intermediate commits, and please fix cosmetics such as
add copyright headers, fix CR/LF, fix file permissions, remove
unnecessary `remote_debugger.py.orig` file, etc.
BTW, the file `remote_wsgi_reload` may be better called
`remote_debugger_wsgi_reloader` so that the files share the same
namespace-like prefix for easier orientation, or else it may be moved
to the WebStyle module and live under a name like
`webinterface_handler_wsgi_reloader` perhaps. (Also, please
acknowledge upstream source in the global THANKS file; we could/should
also commit it under upstream author's name, if possible, like we did
for pep8 or logicutils.)
--
Ticket URL: <http://invenio-software.org/ticket/570#comment:1>
Invenio <http://invenio-software.org>
