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 - Need to rename certain environment variables
Summary: Need to rename certain environment variables
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: other (any) (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other Other
: P1 blocker
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2012-02-16 14:09 CST by Darrell
Modified: 2012-10-19 15:14 CDT (History)
2 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
Patch for updating KDE_FORK_SLAVES (5.02 KB, patch)
2012-02-17 18:59 CST, Darrell 		Details | Diff
Patch for updating KDE_HOME_READONLY (2.06 KB, patch)
2012-02-17 18:59 CST, Darrell 		Details | Diff
Patch for updating KDE_NO_IPV6 (2.62 KB, patch)
2012-02-17 19:00 CST, Darrell 		Details | Diff
Patch for updating KDE_IS_PRELINKED (1.35 KB, patch)
2012-02-17 19:00 CST, Darrell 		Details | Diff
Patch for updating KDE_UTF8_FILENAMES (2.47 KB, patch)
2012-02-17 19:01 CST, Darrell 		Details | Diff
Patch for updating KDE_EXTRA_FSYNC (454 bytes, patch)
2012-02-17 19:09 CST, Darrell 		Details | Diff
Patch for updating KDE_DEBUG (4.08 KB, patch)
2012-02-17 19:20 CST, Darrell 		Details | Diff
Patch for updating KDE_PAM_ACTION (429 bytes, patch)
2012-02-17 19:25 CST, Darrell 		Details | Diff
Patch for updating KCHECKPASS_USER (422 bytes, patch)
2012-02-17 19:38 CST, Darrell 		Details | Diff
Patch for updating KDE_DOUNLOAD (1.02 KB, patch)
2012-02-17 19:39 CST, Darrell 		Details | Diff
Patch for updating some remnant KDEDIR references (14.54 KB, patch)
2012-02-17 20:01 CST, Darrell 		Details | Diff
Patch for updating some remnant KDE_FULL_SESSION references (2.72 KB, patch)
2012-02-17 20:07 CST, Darrell 		Details | Diff
Patch for updating KDEBINBDIR (4.13 KB, patch)
2012-02-17 20:18 CST, Darrell 		Details | Diff
Patch for updating KDE_NOUNLOAD (572 bytes, patch)
2012-02-17 20:23 CST, Darrell 		Details | Diff
Patch for updating KDE_NO_SRV (582 bytes, patch)
2012-02-17 20:27 CST, Darrell 		Details | Diff
Patch for updating KDE_MALLOC (11.17 KB, patch)
2012-02-17 20:35 CST, Darrell 		Details | Diff
Patch for updating KDE_KIOSK_NO_RESTRICTIONS (2.13 KB, patch)
2012-02-17 20:49 CST, Darrell 		Details | Diff
Patch for updating KDE_KIOSK_NO_PROFILES (1.26 KB, patch)
2012-02-17 20:49 CST, Darrell 		Details | Diff
Patch for updating KDE_SLAVE_* (1.92 KB, patch)
2012-02-17 21:07 CST, Darrell 		Details | Diff
Patch for updating KDE_DISPLAY (845 bytes, patch)
2012-02-17 21:13 CST, Darrell 		Details | Diff
Patch for updating KDE_USE_IDN (661 bytes, patch)
2012-02-17 21:16 CST, Darrell 		Details | Diff
Patch for updating KDESYCOCA (3.67 KB, patch)
2012-02-17 21:27 CST, Darrell 		Details | Diff
Patch to update KDESRCDIR (11.59 KB, patch)
2012-02-17 21:37 CST, Darrell 		Details | Diff
{atch for updating KDE_LANG (2.80 KB, patch)
2012-02-17 22:23 CST, Darrell 		Details | Diff
Patch for updating tde-i18n docbook files (33.55 KB, patch)
2012-02-17 23:16 CST, Darrell 		Details | Diff
libtqt: Patch to rename certain environment variables (11.86 KB, patch)
2012-02-27 07:57 CST, Darrell 		Details | Diff
kiosktool: Patch to rename certain environment variables (1.51 KB, patch)
2012-02-27 07:58 CST, Darrell 		Details | Diff
kymoney: Patch to rename certain environment variables (1.46 KB, patch)
2012-02-27 07:58 CST, Darrell 		Details | Diff
tdepim: Patch to rename certain environment variables (6.96 KB, patch)
2012-02-27 07:58 CST, Darrell 		Details | Diff
tdelibs: Patch to rename certain environment variables (29.57 KB, patch)
2012-02-27 07:59 CST, Darrell 		Details | Diff
tdebase: Patch to rename certain environment variables (10.05 KB, patch)
2012-02-27 07:59 CST, Darrell 		Details | Diff
arts: Patch to rename certain environment variables (1.34 KB, patch)
2012-02-27 08:00 CST, Darrell 		Details | Diff
tdeutils: Patch to rename certain environment variables (593 bytes, patch)
2012-02-27 08:00 CST, Darrell 		Details | Diff
tdesdk: Patch to rename certain environment variables (7.33 KB, patch)
2012-02-27 08:00 CST, Darrell 		Details | Diff
libreoffice: Patch to rename certain environment variables (1.74 KB, patch)
2012-02-27 08:01 CST, Darrell 		Details | Diff
tdenetwork: Patch to rename certain environment variables (4.13 KB, patch)
2012-02-27 08:01 CST, Darrell 		Details | Diff
tdesudo: Patch to rename certain environment variables (395 bytes, patch)
2012-02-27 08:02 CST, Darrell 		Details | Diff
kpilot: Patch to rename certain environment variables (517 bytes, patch)
2012-02-27 08:03 CST, Darrell 		Details | Diff
tdelibs: Patch to rename certain environment variables (updated) (32.03 KB, patch)
2012-02-29 21:49 CST, Darrell 		Details | Diff
Show Obsolete (25) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.