View Issue Details

IDProjectCategoryView StatusLast Update
0005695Talerotherpublic2021-08-24 14:23
ReporterMarcello Stanisci Assigned ToMS Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PrioritynormalSeveritytweakReproducibilityhave not tried
Status closedResolutionfixed 
Target Version0.8Fixed in Version0.8 
Summary0005695: talerconfig.py too much susceptible to missed files / directories
DescriptionThe Python talerconfig "copylib" module uses to crash when it fails to find the directory for defaults.
Instead it should just move on and look - maybe print a WARNING - for the next "element".

  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/marcello/prog/taler/bank/talerbank/settings.py", line 24, in <module>
    TC = TalerConfig.from_file(os.environ.get("TALER_CONFIG_FILE"))
  File "/home/marcello/prog/taler/bank/talerbank/talerconfig.py", line 348, in from_file
    cfg.load_defaults()
  File "/home/marcello/prog/taler/bank/talerbank/talerconfig.py", line 409, in load_defaults
    self.load_dir(os.path.join(prefix, "share/taler/config.d"))
  File "/home/marcello/prog/taler/bank/talerbank/talerconfig.py", line 436, in load_dir
    files = os.listdir(dirname)
NotADirectoryError: [Errno 20] Not a directory: '/dev/null/share/taler/config.d'
Makefile:882: recipe for target 'check' failed
make: *** [check] Error 1
Steps To ReproduceCheckout the bank.git, then

./bootstrap

./configuure --prefix=/dev/null

make check
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

buckE

2020-05-11 07:09

reporter   ~0015879

Actually what happens is:

$ ./configure --prefix=/dev/null
./configure: unrecognized option '--prefix=/dev/null'

buckE

2020-06-19 07:42

reporter   ~0016322

Please confirm this is a current bug. I have this:


~/code/bank$ ./configure --prefix=/dev/null
./configure: unrecognized option '--prefix=/dev/null'

MS

2020-09-07 10:16

reporter   ~0016896

This was fixed a while ago. That happens when the config module doesn't find a file/directory:

2020-09-07 12:08:31,242 talerconfig WARNING no base directory found
2020-09-07 12:08:31,242 talerconfig ERROR Configuration file (/not/exist) not found

Issue History

Date ModifiedUsernameFieldChange
2019-04-19 13:43Marcello StanisciNew Issue
2020-04-01 14:28Christian GrothoffAssigned To => buckE
2020-04-01 14:28Christian GrothoffStatusnew => assigned
2020-05-11 07:09buckENote Added: 0015879
2020-06-19 07:42buckENote Added: 0016322
2020-06-19 07:42buckEAssigned TobuckE => Marcello Stanisci
2020-09-04 12:38MSAssigned ToMarcello Stanisci => MS
2020-09-05 11:53Christian GrothoffSeverityminor => tweak
2020-09-07 10:16MSNote Added: 0016896
2020-09-07 10:16MSStatusassigned => informing
2020-09-07 10:16MSResolutionopen => fixed
2020-10-03 12:08Christian GrothoffFixed in Version => 0.8
2020-10-03 12:10Christian GrothoffTarget Version => 0.8
2021-08-24 14:23Christian GrothoffStatusinforming => closed