View Issue Details

IDProjectCategoryView StatusLast Update
0006638Talerobsolete componentpublic2021-08-24 14:23
ReporterFlorian Dold Assigned ToFlorian Dold 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
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version0.8 
Target Version0.8Fixed in Version0.8 
Summary0006638: bank build system issues
DescriptionOn installing taler on a completely fresh system, I've noticed the following issues in the bank:

1. "make install" requires django to be installed at build time. Is this necessary? I'm wondering if we need to do this i18n step for *every* installation. Could this be a separate step that is not automatically executed on every "make install"? If it's really necessary, we should at least check for it and give a nice error message. Right now we get some import error stack trace.

2. The package seems to require "taler-util>=0.8.0", but pip doesn't have this version yet.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

MS

2020-11-03 12:35

reporter   ~0017089

Last edited: 2020-11-03 12:37

Point 1 got addressed here: 915bca1..1a2031b.

The README has also been updated, but in a nutshell: when you want to install the MO files too, you need the two steps:

#1
$ make i18n

#2
$ make install

Christian Grothoff

2020-11-03 18:22

manager   ~0017092

Why would I NOT want to install the .mo files? Usually 'make i18n' is part of make install for standard GNU build systems. I see no good reason to not include 'i18n' as a subtarget of 'install'.

MS

2020-11-03 19:45

reporter   ~0017093

Found this: the alternative is to (1) install all the dependencies before the bank package itself -> (2) generate the .mo files -> (3) install only the bank including the fresh .mo files.

(1) can be done with: "pip3 install --editable ."
(3) with: "pip3 install --no-dependencies ."
(2) is the usual "compilemessages" that needs Django.

MS

2020-11-03 20:30

reporter   ~0017095

This is done here: 1a2031b..f20afac

MS

2020-11-03 20:45

reporter   ~0017096

taler-util 0.8.0 is on PyPI now.

Florian Dold

2020-11-03 20:55

manager   ~0017097

It's rather non-obvious what the --editable does here and why it's needed. Could you document/explain that?

I also don't understand our installation process anymore ... it looks like now

  make install

won't make sure anymore that the run-time dependencies are also installed?

Florian Dold

2020-11-16 11:56

manager   ~0017143

We're using poetry now, which handles dev dependencies well. The pip upload was also fixed by MS.

Issue History

Date ModifiedUsernameFieldChange
2020-11-02 23:29Florian DoldNew Issue
2020-11-02 23:29Florian DoldStatusnew => assigned
2020-11-02 23:29Florian DoldAssigned To => MS
2020-11-03 00:09Florian DoldProduct Version => 0.8
2020-11-03 00:09Florian DoldTarget Version => 0.8
2020-11-03 12:35MSNote Added: 0017089
2020-11-03 12:37MSNote Edited: 0017089
2020-11-03 18:22Christian GrothoffNote Added: 0017092
2020-11-03 19:45MSNote Added: 0017093
2020-11-03 20:30MSNote Added: 0017095
2020-11-03 20:45MSNote Added: 0017096
2020-11-03 20:46MSStatusassigned => informing
2020-11-03 20:46MSResolutionopen => fixed
2020-11-03 20:55Florian DoldNote Added: 0017097
2020-11-03 21:24Florian DoldStatusinforming => assigned
2020-11-06 08:08Christian GrothoffTarget Version0.8 => 0.8.1
2020-11-16 11:56Florian DoldAssigned ToMS => Florian Dold
2020-11-16 11:56Florian DoldStatusassigned => informing
2020-11-16 11:56Florian DoldNote Added: 0017143
2021-07-30 11:57Christian GrothoffFixed in Version => 0.8
2021-07-30 12:02Christian GrothoffTarget Version0.8.1 => 0.8
2021-08-24 14:23Christian GrothoffStatusinforming => closed
2022-08-23 18:26Christian GrothoffCategorybank (demonstrator) => py bank (demonstrator, obsolete)
2023-12-03 00:23Christian GrothoffCategorypy bank (demonstrator, obsolete) => obsolete componet
2023-12-11 19:08Florian DoldCategoryobsolete componet => obsolete component