[Dev] [PATCH] README.maintenance: clarify how to become redmine admin

bill-auger bill-auger at peers.community
Sat Jun 13 19:28:33 GMT 2020


On Sat, 13 Jun 2020 15:26:11 +0200 Denis wrote:
> On Fri, 12 Jun 2020 19:51:19 -0400
> Megver83 <megver83 at hyperbola.info> wrote:
> > wouldn't be a better option to run that script with `sudo -u
> > labs ./enter-rails-console`?  
> Indeed, I'll resend a patch with that improvement.

i dont consider the patch to be an improvement; just because it
is making the process more complex - i could have put the
verbose commands in that README originally - as a matter of
abstraction, i added the HACKERS dict to make the process simpler
and easier to remember

that is defined here:

https://git.parabola.nu/server/redmine.git/tree/config/initializers/constants.rb?h=parabola#n127

if HACKERS['gnutoo'] returned nil, that indicates a bug in the
HACKERS code, or in libretools, where it gets the data from

a real improvement here, would be to simplify the process even
more, with a script which does the entire job - instead of
./enter-rails-console plus the manual typing, one could simply
run ./make-me-web-admin 'gnutoo'; which abstracts the entire
procedure

it was just a matter of the usual trade-off of "how much code to
write" vs "how many caveats to document", and "how much time
should i spend on either"


More information about the Dev mailing list