Ticket #1568 (closed defect: fixed)

Opened 6 years ago

Last modified 5 years ago

Three reporting issues with new delete

Reported by: jweiss Owned by:
Priority: normal Milestone: The Distant Future
Component: -- Keywords:
Cc: jik@… Fixed in version: debathena-delete 10.1.1-0debathena1
Upstream bug:

Description

The new python based version of delete has at least three changes in reporting from the old version:

1) When delete is unable to delete a file it fails to report which file it failed to delete (potentially important if you're deleting multiple files and one fails) The old version used to say something like:

delete: foo: No such file or directory

for each file foo that failed to delete.

2) Same problem in reverse with undelete.

3) undelete is now verbose about the list of files it is undeleting, where as the old version only reported problems.

Change History

comment:1 in reply to: ↑ description Changed 6 years ago by jdreed

  • Status changed from new to review

Replying to jweiss:

The new python based version of delete has at least three changes in reporting from the old version:

1) When delete is unable to delete a file it fails to report which file it failed to delete (potentially important if you're deleting multiple files and one fails) The old version used to say something like:

2) Same problem in reverse with undelete.

 https://github.com/mit-athena/delete/pull/3

3) undelete is now verbose about the list of files it is undeleting, where as the old version only reported problems.

WONTFIX, this is now defined as a feature.

comment:2 Changed 6 years ago by jdreed

  • Status changed from review to committed

committed  245e7c35522f4ddf33a2fabee157c2aae133e864 (Don't suck at Boolean logic in perror()) to master

comment:3 Changed 6 years ago by kaduk

  • Cc jik@… added

comment:4 Changed 5 years ago by andersk

  • Status changed from committed to development
  • Fixed in version set to debathena-delete 10.1.1-0debathena1

comment:5 Changed 5 years ago by andersk

  • Status changed from development to proposed

comment:6 Changed 5 years ago by andersk

  • Status changed from proposed to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.