Description: Hi, I use --with-build-version='$(DEB_VERSION)' \ in my packages to also display the Debian package version in About. Worked until 6.4.5, you get: Build-ID: 1:6.4.5-1 Looks like the old dialog got it from the info of the system: $ grep 6.1.5 /usr/lib/libreoffice/program/*rc /usr/lib/libreoffice/program/versionrc:BuildVersion=1:6.1.5-3+deb10u6 That one is still there in 7.0.0: # cat /usr/lib/libreoffice/program/versionrc [Version] AllLanguages=en-US af am ar as ast be bg bn br bs ca ca-valencia cs cy da de dz el en-GB en-ZA eo es et eu fa fi fr ga gd gl gu gug he hi hr hu id is it ja ka kk km kmr-Latn kn ko lt lv mk mn ml mr nb ne nl nn nr nso oc om or pa-IN pl pt pt-BR ro ru rw si sk sl sr sr-Latn ss st sv szl ta te tg th tn tr ts ug uk uz ve vi xh zh-CN zh-TW zu BuildVersion=1:7.0.0~rc1-3 [...] but oesn't appear in the dialog anymore. Regards, Rene P.S.: What is the difference between that long-existing --with-build-version and the (new?) --with-extra-buildid? Should one use --with-extra-buildid? That seems to be honoured in cui directly. (Then --with-build-version should have been removed already.) Steps to Reproduce: - Actual Results: - Expected Results: - Reproducible: Always User Profile Reset: No Additional Info: -
(if that one's fixed, please backport it to libreoffice-7-0)
Cannot see where this should have been lost in [1], QA please do your magic. To me it's not so clear whether we should keep the -with-build-version switch (and show it consequently) or remove it. Cloph, what's your opinion? [1] https://gerrit.libreoffice.org/c/core/+/91908
Created attachment 162658 [details] dialog comparison
Created attachment 162662 [details] --with-build-version vs. --with-extra-buildid Hmm, ok, --with-extra-buildid does what it does if I add "Debian package version" (or so) there. (see new screenshot.) So if it was intended that --with-build-version is not honoured that might go away...
IIRC I removed it from the Ui to save space. The build ID is still in the copied information. But your comment 4 is a good question.
Discussed the topic in the ESC meeting today and there was no concerns to remove one of the two flags.
No, this is not an enhancement. Right now we have a ./configure option doing nothing. This is not an enhancement but a bug.
https://gerrit.libreoffice.org/c/core/+/98923
(In reply to Rene Engelhard from comment #7) > Right now we have a ./configure option doing nothing. That's wrong. The information is shown in the copied text.
OK, maybe, but it is not visible so people will not see it, as shown in my screenshots. I'd for sure not expect it being there when one doesn't see it.
Rene Engelhard committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/5fdf2009d21fa220dfee70ea755bd698c16257a7 tdf#134522 remove --with-build-version ./configure flag It will be available in 7.1.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Rene Engelhard committed a patch related to this issue. It has been pushed to "libreoffice-7-0": https://git.libreoffice.org/core/commit/a2918297c1d105f0ab7b52f28fc325af6440394b tdf#134522 remove --with-build-version ./configure flag It will be available in 7.0.1. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Should this be closed or if not, why?
Let's consider this fixed, Rene if you have objections, please reopen.