Project

General

Profile

Bug #12573

Synchronize the defaults for and spelling of tmpdir

Added by Rob Kutschke over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Low
Category:
Application
Target version:
Start date:
05/09/2016
Due date:
% Done:

100%

Estimated time:
2.00 h
Spent time:
Occurs In:
Scope:
Internal
Experiment:
Mu2e
SSI Package:
art
Duration:

Description

This is a low priority request. This report is for art 1_17_07.

For RootOutput the default for tmpDir is . but for TFileService the default is to write it in the same directory that the final output file will appear in. It would be easier to document and support if these two options had the same default. My vote is that the default of RootOutput should be changed to match that of TFileService. Are there any other modules/services that also have a tmpdir policy?

For both RootOutput and TFileService the parameter set parameter that controls the tmp directory is named tmpDir, with a capital D. The command line argument (which overrides both defaults) is spelled --tmpdir, with a lower case D. If it's not too late the spelling should be synchronized.

Associated revisions

Revision c3368287
Added by Christopher Green over 3 years ago

Implement issue #12573.

History

#1 Updated by Kyle Knoepfel over 3 years ago

  • Status changed from New to Accepted
  • Assignee set to Kyle Knoepfel
  • Estimated time set to 2.00 h
  • SSI Package art added
  • SSI Package deleted ()

We will make sure the behavior is consistent between RootOutput and TFileService, and synchronize the spelling.

#2 Updated by Christopher Green over 3 years ago

  • Category set to Application
  • Status changed from Accepted to Resolved
  • Assignee changed from Kyle Knoepfel to Christopher Green
  • Target version set to 2.00.01
  • % Done changed from 0 to 100

Implemented with c336828: --tmpDir is now a synonym for --tmpdir, and RootOutput now has the same default value for tmpDir as TFileService, that of the containing directory of fileName.

#3 Updated by Kyle Knoepfel over 3 years ago

  • Status changed from Resolved to Closed


Also available in: Atom PDF