truncate version number in durexforth.prg #586
Replies: 4 comments
-
Another option: \ file "version.fs"
: version s" v5.1.0-whatever" ; \ towards the end of "base.fs"
include version
:noname version type cr ; start ! |
Beta Was this translation helpful? Give feedback.
-
I suppose the option above doesn't solve this for everything defined after Hm. Besides truncating, you might also pad to a certain maximum size. |
Beta Was this translation helpful? Give feedback.
-
I realized also without the tech perspective, I think it looks nicer with
plain vN.N.N in the startup message.
sön 22 dec. 2024 kl. 18:27 skrev ekipan ***@***.***>:
… for changes that are not code related. This is not so nice when debugging.
I suppose the option above doesn't solve this for everything defined after
version, just before it.
Hm. Besides truncating, you might also pad to a certain maximum size.
—
Reply to this email directly, view it on GitHub
<#586 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAY34O2EJZNQW7IOLQ5S7IT2G3ZBNAVCNFSM6AAAAABUBPNCN2VHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTCNRUGM4DEMA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
In the startup, you should go with what looks nice and the benefit to debug address stability. |
Beta Was this translation helpful? Give feedback.
-
Hallo!
Since #345 , the version number is baked into durexForth, and displayed as a startup message.
I think it is nice to show the version at startup, but it is somewhat impractical, because the length of the version string changes depending on git status. This means the .prg size may change, and addresses jump around in RAM, for changes that are not code related. This is not so nice when debugging.
How would you feel about changing the version string, so that it always follows the format vN.N.N ? (i.e. skip the last digit, and -M). Then the version string will be a lot more stable. The "full" info could still be printed in disk listing.
Beta Was this translation helpful? Give feedback.
All reactions