IPP Software Navigation Tools IPP Links Communication Pan-STARRS Links

Opened 19 years ago

Closed 18 years ago

Last modified 18 years ago

#922 closed enhancement (fixed)

For peace of mind, pxadmin should say which database it will be operating on

Reported by: jester@… Owned by: jhoblitt
Priority: high Milestone:
Component: PanTasks Version: 2.2
Severity: minor Keywords:
Cc:

Description

Every time I run pxadmin, there's a leap of faith involved - am I really deleting my own database, not somebody else's? I'm hoping it's an easy fix to make pxadmin say "delete the chip table of project jester" instead of just "delete the chip table?"

Change History (7)

comment:1 by Paul Price, 19 years ago

Cc: price@… added
Owner: changed from eugene to jhoblitt

comment:2 by eugene, 19 years ago

Cc: eugene@… added
Status: newassigned

perhaps pxadmin should require a -dbname

comment:3 by jester@…, 18 years ago

I second Gene's idea for a -dbname argument. Editing .ipprc every time to run pxadmin is a bit clumsy if I'm working with different databases in parallel.

comment:4 by Paul Price, 18 years ago

pxadmin (and all programs which call pmConfigRead) already takes a "-dbname" (along with "-dbserver", "-dbuser", "-dbpassword" and "-dbport").

Should we alter pxadmin to require (some subset of) these? Or is it sufficient to echo the name of the database we're altering?

comment:5 by eugene, 18 years ago

pxadmin does actually report the dbname.

the standard install has dbname set to XXX in the site.config for safety.

It is reasonable to consider this insufficient for pxadmin, and to require -dbname for it explicitly.

comment:6 by Paul Price, 18 years ago

Resolution: fixed
Status: assignedclosed

Fixed in CVS head: pxadmin now requires "-dbname".
The only place that I can think of that this would be a problem is in ippTasks/simtest.pro, and that already explicitly includes "-dbname".

comment:7 by jester@…, 18 years ago

Paul said:

pxadmin (and all programs which call pmConfigRead) already takes a
"-dbname" (along with "-dbserver", "-dbuser", "-dbpassword" and
"-dbport").

Good to know! I didn't realize this because calling pxadmin without arguments only tells me about -create -delete -recreate as possible arguments. The same thing happens with a lot of other commands, which are only minimally self-documenting. So one thing I'd like to do in September, if time permits, would be to go through all IPP commands and add more --help text. This might answer a few FAQs right there on the command line.

Note: See TracTickets for help on using tickets.