View Issue Details

IDProjectCategoryView StatusLast Update
0005098TalerWeb site(s)public2020-03-31 14:04
ReporterMarcello Stanisci Assigned Tonikita 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 closedResolutionreopened 
Product Versiongit (master) 
Target Version0.7.0Fixed in Version0.7.0 
Summary0005098: --prefix option would be useful
DescriptionGiven that now it is a BB worker which builds the site, it needs to copy all the
HTML/CSS/static files in a separate directory. The woker does that becasue it wipes the local checkout before building the site, and that erases useful files too. So the useful files are to be copied in a safer place.

A './confifure --prefix' option would save the BB worker from manually copying HTML/CSS/static file into the safer place.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Relationships

related to 0005055 closedMarcello Stanisci website updates (www, www-stage, docs) should go into buildbot and not into cron 

Activities

nikita

2019-07-15 17:18

reporter   ~0014683

Ack, will get to this after GSoC.

Florian Dold

2019-08-29 20:55

manager   ~0014823

I don't think --prefix is the right way to do this.

Websites that are served with python are already packaged like normal python packages, either installed globally or per-user. There's nothing more we need to do there.

For websites that have just static files, the build system should produce a "dist" dir that contains all the files, and we then just need to copy this one directory instead of collecting together files manually. No need to complicate things with a --prefix argument and an "install" process that just copies data.

nikita

2019-09-02 07:54

reporter   ~0014833

Okay, this is what I did with gnunet.org

nikita

2019-09-03 14:55

reporter   ~0014841

I've switched the sh file used in the job for stage to copy file from the directory 'rendered' which contains all files (similar to how gnunet.org should be build once we have a buildbot doing this). Is this what you wanted, Marcello?

Florian Dold

2019-11-01 22:22

manager   ~0015047

I've actually revised my opinion on this issue. The landing.git repository already supports --prefix, and www.git should do so too.

The main reason why I changed opinion on this matter is that we now have the taler-build-scripts.git, which makes setting up the build system to do this much easier and uniform across repositories.

nikita

2019-12-14 12:54

reporter   ~0015155

This is satisfied by recent config.mk and the install rule in www.git as far as I can see.
running make install would mkdir and then cp to the prefix/share/taler-www

Florian Dold

2019-12-14 12:58

manager   ~0015157

Looks good to me.

nikita

2019-12-14 14:45

reporter   ~0015159

It's still not fixed, i need to work in the right path into the Makefile of www. Will close when done.

nikita

2019-12-14 15:04

reporter   ~0015160

instead of "hopping around" for stable/master, why don't we use a git worktree for one of the branches? this way we just have to cd to the directory and get rid of conflicting state (better would be to build in a different way)

Florian Dold

2019-12-14 15:06

manager   ~0015161

Not sure what you mean. We already have to separate checkouts for master and stable. Git's worktrees just allow us to have to branches checked out in some sub-folder of the same cloned repo.

nikita

2019-12-14 15:19

reporter   ~0015162

Okay.
Then deployment really needs better documentation, I'm just going by buildbot logs.

Christian Grothoff

2019-12-25 11:05

manager   ~0015221

Why is this bug still open? Seems to work...

nikita

2020-01-20 14:24

reporter   ~0015292

I haven't done any bug triage recently. Closing as implemented.

Issue History

Date ModifiedUsernameFieldChange
2017-06-29 09:29Marcello StanisciNew Issue
2017-06-29 09:29Marcello StanisciRelationship addedrelated to 0005055
2017-10-23 08:23Christian GrothoffSeverityminor => tweak
2017-10-23 08:23Christian GrothoffProduct Version => git (master)
2017-10-23 08:53Christian GrothoffStatusnew => implementing
2017-12-14 14:58Christian GrothoffTarget Version => 0.7.1
2019-07-15 17:18nikitaNote Added: 0014683
2019-07-15 17:18nikitaAssigned To => nikita
2019-07-15 17:18nikitaStatusimplementing => assigned
2019-08-29 20:55Florian DoldNote Added: 0014823
2019-09-02 07:54nikitaNote Added: 0014833
2019-09-03 14:55nikitaNote Added: 0014841
2019-09-03 14:55nikitaAssigned Tonikita => Marcello Stanisci
2019-11-01 22:22Florian DoldNote Added: 0015047
2019-11-01 22:22Florian DoldAssigned ToMarcello Stanisci => nikita
2019-12-14 12:54nikitaNote Added: 0015155
2019-12-14 12:58Florian DoldStatusassigned => informing
2019-12-14 12:58Florian DoldResolutionopen => fixed
2019-12-14 12:58Florian DoldNote Added: 0015157
2019-12-14 14:45nikitaStatusinforming => feedback
2019-12-14 14:45nikitaResolutionfixed => reopened
2019-12-14 14:45nikitaNote Added: 0015159
2019-12-14 15:04nikitaNote Added: 0015160
2019-12-14 15:06Florian DoldNote Added: 0015161
2019-12-14 15:19nikitaNote Added: 0015162
2019-12-25 11:05Christian GrothoffNote Added: 0015221
2020-01-20 14:24nikitaStatusfeedback => informing
2020-01-20 14:24nikitaNote Added: 0015292
2020-03-17 01:08Christian GrothoffFixed in Version => 0.7.0
2020-03-17 01:08Christian GrothoffTarget Version0.7.1 => 0.7.0
2020-03-31 14:04Christian GrothoffStatusinforming => closed