D12733: The exported version env var should have a consistent length
Ahmad Samir
noreply at phabricator.kde.org
Tue May 8 07:42:11 UTC 2018
ahmadsamir added a comment.
In D12733#259436 <https://phabricator.kde.org/D12733#259436>, @hindenburg wrote:
> I was wondering if .remove(".").append("0").left(6) would be easier.
I see, (sorry I didn't make the connection earlier).
I tested that with a couple of cases, and it wouldn't work, for example:
18.08.1 -> 1808100 -> 180810
18.08.10 -> 18081000 -> 180810
18.08.2 -> 1808200 -> 180820
18.08.20 -> 18082000 -> 180820
and if KF5 ever breaks the double-number release versions, e.g. 100.08.2, :), then that bit of code would fail. I'd rather we make the code just work for all the cases we can think of now.
REPOSITORY
R319 Konsole
REVISION DETAIL
https://phabricator.kde.org/D12733
To: ahmadsamir, #konsole, hindenburg
Cc: herrold, ngraham, hindenburg
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20180508/a577f1c9/attachment-0001.html>
More information about the konsole-devel
mailing list