By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 860

Summary:	Need to rename certain environment variables
Product:	TDE	Reporter:	Darrell <darrella>
Component:	other (any)	Assignee:	Timothy Pearson <kb9vqf>
Status:	RESOLVED FIXED
Severity:	blocker	CC:	bugwatch, darrella
Priority:	P1
Version:	R14.0.0 [Trinity]
Hardware:	Other
OS:	Other
Compiler Version:		TDE Version String:
Application Version:		Application Name:
Attachments:	Patch for updating KDE_FORK_SLAVES Patch for updating KDE_HOME_READONLY Patch for updating KDE_NO_IPV6 Patch for updating KDE_IS_PRELINKED Patch for updating KDE_UTF8_FILENAMES Patch for updating KDE_EXTRA_FSYNC Patch for updating KDE_DEBUG Patch for updating KDE_PAM_ACTION Patch for updating KCHECKPASS_USER Patch for updating KDE_DOUNLOAD Patch for updating some remnant KDEDIR references Patch for updating some remnant KDE_FULL_SESSION references Patch for updating KDEBINBDIR Patch for updating KDE_NOUNLOAD Patch for updating KDE_NO_SRV Patch for updating KDE_MALLOC Patch for updating KDE_KIOSK_NO_RESTRICTIONS Patch for updating KDE_KIOSK_NO_PROFILES Patch for updating KDE_SLAVE_* Patch for updating KDE_DISPLAY Patch for updating KDE_USE_IDN Patch for updating KDESYCOCA Patch to update KDESRCDIR {atch for updating KDE_LANG Patch for updating tde-i18n docbook files libtqt: Patch to rename certain environment variables kiosktool: Patch to rename certain environment variables kymoney: Patch to rename certain environment variables tdepim: Patch to rename certain environment variables tdelibs: Patch to rename certain environment variables tdebase: Patch to rename certain environment variables arts: Patch to rename certain environment variables tdeutils: Patch to rename certain environment variables tdesdk: Patch to rename certain environment variables libreoffice: Patch to rename certain environment variables tdenetwork: Patch to rename certain environment variables tdesudo: Patch to rename certain environment variables kpilot: Patch to rename certain environment variables tdelibs: Patch to rename certain environment variables (updated)

Description Darrell 2012-02-16 14:09:35 CST

To avoid conflicts with KDE3/KDE4 installations, the following environment variables found in the GIT source tree should be renamed:

KDE_FORK_SLAVES
KDE_HOME_READONLY
KDE_NO_IPV6
KDE_IS_PRELINKED
KDE_UTF8_FILENAMES
KDE_EXTRA_FSYNC
KDE_DEBUG
KDE_PAM_ACTION
KDE_DOUNLOAD
KDE_NOUNLOAD
KDE_NO_SRV
KDE_MALLOC
KDE_KIOSK_NO_RESTRICTIONS
KDE_KIOSK_NO_PROFILES
KDE_SLAVE_VALGRIND
KDE_SLAVE_VALGRIND_SKIN
KDE_SLAVE_DEBUG_WAIT
KDE_DISPLAY
KDE_USE_IDN
KDESYCOCA
KDESRCDIR
KDEBINDIR
KDE_LANG
KDE_FULL_SESSION (although renamed in most places, variable still exists in some sources)

A quick perusal of the GIT source tree reveals these variables still exist as provided from KDE3 and have not been renamed. Other than KDE_LANG in various admin m4 files, none of these variables are heavily used and a one-time search-and-replace should suffice.

Originally I found some of these variables being referenced in various help handbook files (docbook source files). I generated a more complete list by searching the source tree for getenv and setenv functions and "KDE."

Comment 1 Darrell 2012-02-17 18:45:03 CST

For the source code files I'm going to submit one patch for each variable.

For the tde-i18n files I'm going to submit one large patch for all variables, which are all docbook files.

Comment 2 Darrell 2012-02-17 18:59:10 CST

Created attachment 363 [details]
Patch for updating KDE_FORK_SLAVES

Comment 3 Darrell 2012-02-17 18:59:39 CST

Created attachment 364 [details]
Patch for updating KDE_HOME_READONLY

Comment 4 Darrell 2012-02-17 19:00:06 CST

Created attachment 365 [details]
Patch for updating KDE_NO_IPV6

Comment 5 Darrell 2012-02-17 19:00:35 CST

Created attachment 366 [details]
Patch for updating KDE_IS_PRELINKED

Comment 6 Darrell 2012-02-17 19:01:00 CST

Created attachment 367 [details]
Patch for updating KDE_UTF8_FILENAMES

Comment 7 Darrell 2012-02-17 19:09:34 CST

Created attachment 368 [details]
Patch for updating KDE_EXTRA_FSYNC

Comment 8 Darrell 2012-02-17 19:20:43 CST

Created attachment 369 [details]
Patch for updating KDE_DEBUG

Comment 9 Darrell 2012-02-17 19:25:58 CST

Created attachment 370 [details]
Patch for updating KDE_PAM_ACTION

Comment 10 Darrell 2012-02-17 19:38:50 CST

Created attachment 371 [details]
Patch for updating KCHECKPASS_USER

Comment 11 Darrell 2012-02-17 19:39:31 CST

Created attachment 372 [details]
Patch for updating KDE_DOUNLOAD

Comment 12 Darrell 2012-02-17 20:01:00 CST

Created attachment 373 [details]
Patch for updating some remnant KDEDIR references

Comment 13 Darrell 2012-02-17 20:07:24 CST

Created attachment 374 [details]
Patch for updating some remnant KDE_FULL_SESSION references

Comment 14 Darrell 2012-02-17 20:18:17 CST

Created attachment 375 [details]
Patch for updating KDEBINBDIR

Comment 15 Darrell 2012-02-17 20:23:52 CST

Created attachment 376 [details]
Patch for updating KDE_NOUNLOAD

Comment 16 Darrell 2012-02-17 20:27:19 CST

Created attachment 377 [details]
Patch for updating KDE_NO_SRV

Comment 17 Darrell 2012-02-17 20:35:46 CST

Created attachment 378 [details]
Patch for updating KDE_MALLOC

Comment 18 Darrell 2012-02-17 20:49:22 CST

Created attachment 379 [details]
Patch for updating KDE_KIOSK_NO_RESTRICTIONS

Comment 19 Darrell 2012-02-17 20:49:53 CST

Created attachment 380 [details]
Patch for updating KDE_KIOSK_NO_PROFILES

Comment 20 Darrell 2012-02-17 21:07:38 CST

Created attachment 381 [details]
Patch for updating KDE_SLAVE_*

Comment 21 Darrell 2012-02-17 21:13:49 CST

Created attachment 382 [details]
Patch for updating KDE_DISPLAY

Comment 22 Darrell 2012-02-17 21:16:32 CST

Created attachment 383 [details]
Patch for updating KDE_USE_IDN

Comment 23 Darrell 2012-02-17 21:27:26 CST

Created attachment 384 [details]
Patch for updating KDESYCOCA

Comment 24 Darrell 2012-02-17 21:37:45 CST

Created attachment 385 [details]
Patch to update KDESRCDIR

Comment 25 Darrell 2012-02-17 22:23:22 CST

Created attachment 386 [details]
{atch for updating KDE_LANG

This patch only updates source code references to the environment variable KDE_LANG and does not modify any automake files.

Comment 26 Darrell 2012-02-17 23:16:35 CST

Created attachment 387 [details]
Patch for updating tde-i18n docbook files

This patch only updates the environment variable references in the tde-i18n docbook files.

Comment 27 Darrell 2012-02-25 00:47:56 CST

Now that I have pushed some patches in GIT I see that these patches won't work. All of them need to done on a module basis. :(

Comment 28 Darrell 2012-02-25 13:11:43 CST

I now have these patches reorganized according to module.

Comment 29 Darrell 2012-02-27 07:57:34 CST

Created attachment 415 [details]
libtqt: Patch to rename certain environment variables

Comment 30 Darrell 2012-02-27 07:58:00 CST

Created attachment 416 [details]
kiosktool: Patch to rename certain environment variables

Comment 31 Darrell 2012-02-27 07:58:26 CST

Created attachment 417 [details]
kymoney: Patch to rename certain environment variables

Comment 32 Darrell 2012-02-27 07:58:53 CST

Created attachment 418 [details]
tdepim: Patch to rename certain environment variables

Comment 33 Darrell 2012-02-27 07:59:15 CST

Created attachment 419 [details]
tdelibs: Patch to rename certain environment variables

Comment 34 Darrell 2012-02-27 07:59:42 CST

Created attachment 420 [details]
tdebase: Patch to rename certain environment variables

Comment 35 Darrell 2012-02-27 08:00:05 CST

Created attachment 421 [details]
arts: Patch to rename certain environment variables

Comment 36 Darrell 2012-02-27 08:00:31 CST

Created attachment 422 [details]
tdeutils: Patch to rename certain environment variables

Comment 37 Darrell 2012-02-27 08:00:56 CST

Created attachment 423 [details]
tdesdk: Patch to rename certain environment variables

Comment 38 Darrell 2012-02-27 08:01:26 CST

Created attachment 424 [details]
libreoffice: Patch to rename certain environment variables

Comment 39 Darrell 2012-02-27 08:01:48 CST

Created attachment 425 [details]
tdenetwork: Patch to rename certain environment variables

Comment 40 Darrell 2012-02-27 08:02:26 CST

Created attachment 426 [details]
tdesudo: Patch to rename certain environment variables

Comment 41 Darrell 2012-02-27 08:03:20 CST

Created attachment 427 [details]
kpilot: Patch to rename certain environment variables

Comment 42 Darrell 2012-02-29 21:49:45 CST

Created attachment 450 [details]
tdelibs: Patch to rename certain environment variables (updated)

Comment 43 Darrell 2012-03-04 13:04:03 CST

I have been using these patches for the following modules in my daily builds:

dependencies/arts/arts-env-vars.diff
tdebase/tdebase-env-vars.diff
tdelibs/tdelibs-env-vars.diff
tdenetwork/tdenetwork-env-vars.diff
tdepim/tdepim-env-vars.diff
tdesdk/tdesdk-env-vars.diff
tdeutils/tdeutils-env-vars.diff

I also have patches for the following that I have not yet tested:

applications/kiosktool/kiosktool-env-vars.diff
applications/kmymoney/kmymoney-env-vars.diff
applications/kpilot/kpilot-env-vars.diff
applications/tdesudo-env-vars.diff
libraries/libtqt-perl/libtqt-perl-env-vars.diff
other/libreoffice/libreoffice-env-vars.diff

Should I push these patches?

Comment 44 Darrell 2012-03-07 19:35:09 CST

tdelibs: merged in GIT hash 691a81483430c4b7b4433d45381e8d32b6d56909

tdebase: merged in GIT hash f62ebdf09ec164b504966b5cb648d562353c0859

tdepim: merged in GIT hash 7ad5bf874d0d057c92b44527be6d309f07bc9a70

tdeutils: merged in GIT hash 96adcf5659a0a56736d29a14412c920b46917f41

tdenetwork: merged in GIT hash 1c87b8cb34db04f80af3755ed9769f1a074386e1

tdesdk: merged in GIT hash 497ee97a3c67c75c38bf9e411ef50fa8207c50f7

kiosktool: merged in GIT hash 64998f1f766e383b6d5d0d2368c8158b49934270

tdesudo: merged in GIT hash 906dafb1ffa93d585100eb1661dc28e8f61cc100

kymoney: merged in GIT hash 0f1adfd1356d736b0f166ea85960459630a16492

kpilot: merged in GIT hash 3bef4e0b74764e2a0ddb401abf469e953fc75fd5

arts: merged in GIT hash d4e7aaa4210a56b81bc99122a7c47f6a6f552265

libtqt: merged in GIT hash 4a9e3e14f67e172352fe81d0e6307cb3986135a9

This resolves the bug report.