I had problems in Slackware with the 3.5.13 startkde script. Biggest problem was starting KDE 4 apps on its own because $KDEDIRS was explicitly declared in the script and hard-coded to include /usr/share. That variable is intended to be user defined and should not be hard-coded in Trinity.
Second problem was duplication of several snippets of code.
Third problem was snippets of code that never got executed because a previous condition was satisfied.
Fourth problem was the $XDG_DATA_DIRS path. I raised this issue in a previous post. I want the option of whether KDE4 apps appear in my Trinity menu. I can solve that problem easily in Slackware for both types of people using /etc/profile.d scripts. I don't know how other distros would handle that when there is no support for /etc/profile.d. I don't think this variable should be hard-coded in startkde unless completely undeclared.
So I added conditions to set these variables only if they don't include Trinity paths.
I don't want to submit this (yet) as a bug report. I would be grateful if some of you would test this new script in your distro.
I suspect we need more work with this script in order to function correctly in all distros. We probably need a lot of conditional tests.
Here are links to an updated script --- a patch and full script.
http://humanreadable.nfshost.com/trinity/patches/kdebase/startkde.diff
http://humanreadable.nfshost.com/trinity/patches/kdebase/startkde.new
The patch allows you to quickly see the changes and the full script is there for easier copying to your system.
Darrell
On Thu, Nov 17, 2011 at 6:25 PM, Darrell Anderson humanreadable@yahoo.comwrote:
I had problems in Slackware with the 3.5.13 startkde script. Biggest problem was starting KDE 4 apps on its own because $KDEDIRS was explicitly declared in the script and hard-coded to include /usr/share. That variable is intended to be user defined and should not be hard-coded in Trinity.
Second problem was duplication of several snippets of code.
Third problem was snippets of code that never got executed because a previous condition was satisfied.
Fourth problem was the $XDG_DATA_DIRS path. I raised this issue in a previous post. I want the option of whether KDE4 apps appear in my Trinity menu. I can solve that problem easily in Slackware for both types of people using /etc/profile.d scripts. I don't know how other distros would handle that when there is no support for /etc/profile.d. I don't think this variable should be hard-coded in startkde unless completely undeclared.
So I added conditions to set these variables only if they don't include Trinity paths.
I don't want to submit this (yet) as a bug report. I would be grateful if some of you would test this new script in your distro.
I suspect we need more work with this script in order to function correctly in all distros. We probably need a lot of conditional tests.
Here are links to an updated script --- a patch and full script.
http://humanreadable.nfshost.com/trinity/patches/kdebase/startkde.diff
http://humanreadable.nfshost.com/trinity/patches/kdebase/startkde.new
The patch allows you to quickly see the changes and the full script is there for easier copying to your system.
Hi all,
As this been looked into?
Best regards, Tiago
Darrell
To unsubscribe, e-mail: trinity-devel-unsubscribe@lists.pearsoncomputing.net For additional commands, e-mail: trinity-devel-help@lists.pearsoncomputing.net Read list messsages on the Web archive: http://trinity-devel.pearsoncomputing.net/ Please remember not to top-post: http://trinity.pearsoncomputing.net/mailing_lists/#top-posting
Hi all, As this been looked into?
I have received no responses. I am going to submit the new startkde (starttde) script through a bug report. I'm not sure how others are able to run TDE without the changes. Here things break real fast when I use the default script. :(
Darrell
On Tue, Nov 22, 2011 at 3:49 PM, Darrell Anderson humanreadable@yahoo.comwrote:
Hi all, As this been looked into?
I have received no responses. I am going to submit the new startkde (starttde) script through a bug report. I'm not sure how others are able to run TDE without the changes. Here things break real fast when I use the default script. :(
Do that please, perhaps it is worth considering in the next QA period for the upcoming release. It seems an appropriate time to test on a few distros.
Best regards, Tiago
Darrell
To unsubscribe, e-mail: trinity-devel-unsubscribe@lists.pearsoncomputing.net For additional commands, e-mail: trinity-devel-help@lists.pearsoncomputing.net Read list messsages on the Web archive: http://trinity-devel.pearsoncomputing.net/ Please remember not to top-post: http://trinity.pearsoncomputing.net/mailing_lists/#top-posting
On Tue, Nov 22, 2011 at 3:49 PM, Darrell Anderson humanreadable@yahoo.comwrote:
Hi all, As this been looked into?
I have received no responses. I am going to submit the new startkde (starttde) script through a bug report. I'm not sure how others are able to run TDE without the changes. Here things break real fast when I use the default script. :(
Do that please, perhaps it is worth considering in the next QA period for the upcoming release. It seems an appropriate time to test on a few distros.
Best regards, Tiago
+1. I would have likely checked it in to GIT already if it wasn't frozen...
Remember that while the GIT tree is frozen the best thing to do is create a bug report and upload the patch, then mark the bug report as PATCHAVAIL. When GIT thaws there will be a large import of all the patches with that flag on the bugtracker.
Tim
I have received no responses. I am going to submit the new startkde (starttde) script through a bug report. I'm not sure how others are able to run TDE without the changes. Here things break real fast when I use the default script. :(
Do that please, perhaps it is worth considering in the next QA period for the upcoming release. It seems an appropriate time to test on a few distros.
Done as soon as I posted the last message. Bug report 675. I attached a full script rather than a patch.
Darrell
I have received no responses. I am going to submit the new startkde (starttde) script through a bug report. I'm not sure how others are able to run TDE without the changes. Here things break real fast when I use the default script. :(
Do that please, perhaps it is worth considering in the next QA period for the upcoming release. It seems an appropriate time to test on a few distros.
Done as soon as I posted the last message. Bug report 675. I attached a full script rather than a patch.
Patches are easier for the TDE developers, as they allow us to see what has been changed at a glance. If a bug report requires a new file upload, then please attach the original file as well as the new one, so that we can generate difference files internally. Otherwise we cannot commit to GIT for fear of accidentally overwriting some other important bit of the file that changed in the meantime.
Tim
Patches are easier for the TDE developers, as they allow us to see what has been changed at a glance. If a bug report requires a new file upload, then please attach the original file as well as the new one, so that we can generate difference files internally. Otherwise we cannot commit to GIT for fear of accidentally overwriting some other important bit of the file that changed in the meantime.
Done!
Darrell
Tks Darrell.
Best regards, Tiago
On Tue, Nov 22, 2011 at 7:42 PM, Darrell Anderson humanreadable@yahoo.comwrote:
I have received no responses. I am going to submit the new startkde (starttde) script through a bug report. I'm not sure how others are able to run TDE without the changes. Here things break real fast when I use the default script. :(
Do that please, perhaps it is worth considering in the next QA period for the upcoming release. It seems an appropriate time to test on a few distros.
Done as soon as I posted the last message. Bug report 675. I attached a full script rather than a patch.
Darrell
To unsubscribe, e-mail: trinity-devel-unsubscribe@lists.pearsoncomputing.net For additional commands, e-mail: trinity-devel-help@lists.pearsoncomputing.net Read list messsages on the Web archive: http://trinity-devel.pearsoncomputing.net/ Please remember not to top-post: http://trinity.pearsoncomputing.net/mailing_lists/#top-posting
-
Darrell Anderson -
Tiago Marques -
Timothy Pearson