SoftVarE-Group / Course-on-Software-Product-Lines

Material for lectures on software product lines
Creative Commons Attribution Share Alike 4.0 International
11 stars 0 forks source link

Build: Paderborn not a valid option for university #21

Open pmbittner opened 4 months ago

pmbittner commented 4 months ago

Hi all,

I followed the instructions for building the slides in the README.md to build the slides for Paderborn. When I run

make 01-introduction.tex university=paderborn

I get the message make: Nothing to be done for '01-introduction.tex'. but there is no pdf. I checked the makefile and it seems that the feature university does not yet allow the value paderborn.

Also, there are two other build scripts compile_ulm.sh and compile_paderborn.sh. These scripts are undocumented. Are they orthogonal to the makefile? If so, why are there two build systems? If not, when and how to use these scripts? Does make call them? Running

./compile_paderborn.sh

gives ERROR: is not a valid integer. Running

./compile_paderborn.sh 1

just prints 01-introduction and does nothing more.

So the problems are:

ekuiter commented 4 months ago

Try make 01-introduction.pdf university=paderborn (file extension pdf, not tex). For me, this works, and make university=paderborn compiles them all. I updated the Makefile to include paderborn as an allowed value.

ekuiter commented 4 months ago

These scripts are undocumented. Yes, @skrieter wrote them, maybe he can explain. The Makefile does not call them, I have only ever used the Makefile and the documented commands in the README. (I am also not sure about the purpose of spl1.tex and spl2.tex. These currently cause the Makefile to stop with an error, but at this point all individual slides should have been compiled.)

pmbittner commented 4 months ago

Try make 01-introduction.pdf university=paderborn (file extension pdf, not tex). For me, this works, and make university=paderborn compiles them all. I updated the Makefile to include paderborn as an allowed value.

Omg I am so dumb. :D Thanks for the hint and thanks for updating the makfile. It works as expected now. :)

ekuiter commented 4 months ago

this actually happens to me all the time, too ^^

ekuiter commented 4 months ago

FYI: There are currently two new padding bugs (#16, #17) due to the template update, which I will try to fix next week.

tthuem commented 4 months ago

@skrieter Can you document how to use your compile scripts in the readme?

ekuiter commented 3 months ago

things to consider:

I suspect it would be best to integrate it fully with the Makefile to avoid confusion. To do this, I would need more information on the script's purpose.