Update configure.ac

[tschoonj]

Ok, ik zie wat er gebeurd is. Zal een en ander fixen.

[tschoonj]

Als je bijvoorbeeld wil verder werken op deze branch, nadat ik de laatste commit heb gepushed, moet je:

git fetch
git checkout version1.0 (indien je nog niet op de branch zou zitten)
git rebase origin/version1.0 (als je uncommitted changes hebt zal je eerst moeten stashen en daarna unstashen)
... do your commits...
git push (als dit niet zou werken is dat misschien omdat ik in tussentijd al nieuwe commits heb gepushed, in dat geval opnieuw git fetch, git rebase origin/version1.0... Gebruik in geen geval git push --force!!!)

[PieterTack]

Oke, probleem zat dus dat ik blijkbaar exact zelfde naam moet checkouten, en dat ik die rebase niet had gedaan. Thx! :)

[tschoonj]

Als ge met twee of meer op 1 branch werkt, moet ge regelmatig fetchen en rebasen, anders wordt het een soep van merge commits 😄

En gebruik in zo een situatie echt nooit git push --force/-f: dat is in deze context altijd verkeerd. Ik weet dat het aantrekkelijk is, maar probeer de verleiding te weerstaan 😄

[PieterTack]

oei, ik deed geen force :( Ik doe altijd gewoon git push

[tschoonj]

oei, ik deed geen force :( Ik doe altijd gewoon git push

👍 houden zo!

[PieterTack]

Kdenk trouwens dat Travis misschien error geeft omdat PACKAGE_VERSION nergens wordt gedefinieerd? Of wordt dat meegegeven door dat nummer in configure.ac?

EDIT: idd, wordt bepaald door AC_INIT dus :) Makes sense.

[tschoonj]

DIe PACKAGE_VERSION zit in config.h, welke gegenereerd wordt nadat ge ./configure hebt gedaan, idd op basis van wat in AC_INIT zit. Je kan die header ook expliciet includen in die file als je wil.

Ik ga trouwens Travis eruit gooien, het kan toch niet overweg met uw langdurige tests, en Github Actions wel...

[PieterTack]

op xmi4 lukken de tests alleszins wel, dus ook version

[tschoonj]

Ik heb nu '-Wall -Werror' opgezet en hij vindt toch nog problemen op de CI. In het geval van die uninitialised j, zou ik de declaratie ervan gewoon tot voor de for loop brengen, het wordt toch niet eerder gebruikt.

[tschoonj]

Ik zal dit laatste probleem even fixen.

Btw, heb gezien dat ge serieus bezig zijt met Python nu! Nice! Wat vindt ge ervan?

[PieterTack]

Thanks!

Python: ja, het is wel plezant :) Vooral plezant dat er zoveel functionaliteit is etc. En die classes met hun eigen defs etc zijn supermachtig al wordt het in het extreme wel moeilijk voor een buitenstaander om de code te 'lezen'. Op vlak van plotten blijf ik IDL stiekem wel eenvoudiger vinden, maar 't is een kwestie van gewenning. En python/matplotlib zijn natuurlijk net iets meer open source, en dus mss nuttiger om skills in te hebben. Nuja, one never forgets their first love ;) Ben nu bezig met Pyside2 (terminal) gui'tje te maken om daaruit mijn labotoestel aansturing uit te voeren.

[tschoonj]

Ik ben klaar hiermee. Ge moogt mergen.

Thanks!

[PieterTack]

Super :)

Is dan de release 'officieel'? Of moeten we nog iets doen om de release te voltooien?

Heb je overigens al eens die examples uit de paper geprobeerd? Zoniet zal ik het zelf eens proberen :) Laszlo ging trouwens ook binnenkort een blik op de paper werpen. Volgende week heeft hij het nog heel druk met (online) examens, maar daarna is zijn schema iets vrijer.

[tschoonj]

Nee een release doe je door een tag te creëren en die dan te pushen naar Github. Ik raad u aan een annotated tag te maken.

Die examples zouden moeten werken, maar ge moogt ze altijd zelf eerst eens proberen 😄

[PieterTack]

Hm, dus ik heb een annotated tag gemaakt, althans locally. Ik kan die zien staan. Maar op github zelf lijkt het niet aangepast. Kan zogenaamd niet pushen of comitten omdat ik geen changes maakte... Moet ik nu echt ergens een file wat veranderen en comitten om die tag ook online te erkennen?

Heb ook al git tag -a v1.0 c08982e geprobeerd, om vorige commit te taggen, maar dat lijkt ie ook niet op te pikken.

[tschoonj]

Hey Pieter,

Er is precies iets misgelopen. Kunt ge eens op Skype komen?