View Issue Details

IDProjectCategoryView StatusLast Update
0005693TalerWeb site(s)public2019-12-20 18:12
ReporterLUG Assigned ToChristian Grothoff 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
PrioritynormalSeverityblockReproducibilityalways
Status closedResolutionfixed 
PlatformLinux x64OSUbuntuOS Version16.04
Product Versiongit (master) 
Target Version0.6Fixed in Version0.6 
Summary0005693: Improve setup documentation for landing page repo
DescriptionI tried following the instructions in landing/demo/README to locally setup the landing page repo for development and did not succeed (couldn't successfully execute make). I'd like to take this as an opportunity to improve the setup documentation for which I'd need some help to figure out how it is intended to work.
One issues seems to be that it is not clear from which directory to run which of the setup commands (the location of the README or the repo root). Another seems to be some error in the way the make environment is constructed.

I now phrased this issue in terms of an improvement. If that doesn't fit your conventions, please let me know or feel free to rename it accordingly.
Steps To ReproduceI'm on c05d4a7888e543bd5a17583a2e69fe3679567e75 and try to build the landing page as instructed by demo/README.
Lines starting with $ indicate executed commands, other lines is the standard output resulting from these commands.

$ cd /home/LUG/Workspace/taler_landing/demo/

$ git submodule init

$ it submodule update --remote
Submodule path 'static/web-common': checked out 'ba4a54c65786f35922616175b644f9db7897c87e'

$ AUTOMAKE="automake --foreign" autoreconf -fiv
autoreconf: 'configure.ac' or 'configure.in' is required

$ cd ..

$ AUTOMAKE="automake --foreign" autoreconf -fiv
autoreconf: Entering directory `.'
autoreconf: configure.ac: not using Gettext
autoreconf: running: aclocal --force
autoreconf: configure.ac: tracing
autoreconf: configure.ac: not using Libtool
autoreconf: running: /usr/bin/autoconf --force
autoreconf: configure.ac: not using Autoheader
autoreconf: running: automake --foreign --add-missing --copy --force-missing
autoreconf: Leaving directory `.'

$ ./configure
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking for tsc... no
checking that generated files are newer than configure... done
configure: creating ./config.status
config.status: creating demo/Makefile
config.status: creating demo/static/Makefile
config.status: creating demo/static/web-common/Makefile

$ make
make: *** No targets specified and no makefile found. Stop.

$ cd demo

$ make
Making all in static
make[1]: Entering directory '/home/LUG/Workspace/taler_landing/demo/static'
Making all in web-common
make[2]: Entering directory '/home/LUG/Workspace/taler_landing/demo/static/web-common'
python3 -m jsmin taler-wallet-lib.js > taler-wallet-lib.min.js
/home/LUG/anaconda3/bin/python3: No module named jsmin
Makefile:456: recipe for target 'taler-wallet-lib.min.js' failed
make[2]: *** [taler-wallet-lib.min.js] Error 1
make[2]: Leaving directory '/home/LUG/Workspace/taler_landing/demo/static/web-common'
Makefile:295: recipe for target 'all-recursive' failed
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory '/home/LUG/Workspace/taler_landing/demo/static'
Makefile:347: recipe for target 'all-recursive' failed
make: *** [all-recursive] Error 1
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

LUG

2019-04-19 10:07

reporter   ~0014302

I just saw 0005687 and the deployment/doc/onboarding.texi file, which also describes setting up the landing page repo. Would it make sense to remove move landing/demo/README to landing/README and just refer to the deployment repo for documentation if that is your central source of information regarding setup?

Christian Grothoff

2019-04-20 20:33

manager   ~0014323

c05d4a7..be527b1 moves the README to the top-level and modifies the build system so one can run 'make' from the top level. I also documented the dependencies, including python3-jsmin.

Issue History

Date ModifiedUsernameFieldChange
2019-04-19 09:57LUGNew Issue
2019-04-19 10:07LUGNote Added: 0014302
2019-04-20 13:02Christian GrothoffAssigned To => Christian Grothoff
2019-04-20 13:02Christian GrothoffStatusnew => assigned
2019-04-20 13:02Christian GrothoffProduct Version => git (master)
2019-04-20 13:02Christian GrothoffTarget Version => 0.6
2019-04-20 20:33Christian GrothoffNote Added: 0014323
2019-04-20 20:33Christian GrothoffStatusassigned => informing
2019-04-20 20:33Christian GrothoffResolutionopen => fixed
2019-04-20 20:33Christian GrothoffFixed in Version => 0.6
2019-12-20 18:12Christian GrothoffStatusinforming => closed