Merge lp:~bryanfritt/compiz/compiz-decorator_script-edit_1192376 into lp:compiz/0.9.11
- compiz-decorator_script-edit_1192376
- Merge into 0.9.11
Status: | Needs review | ||||
---|---|---|---|---|---|
Proposed branch: | lp:~bryanfritt/compiz/compiz-decorator_script-edit_1192376 | ||||
Merge into: | lp:compiz/0.9.11 | ||||
Diff against target: |
378 lines (+128/-107) 3 files modified
debian/patches/ubuntu-config.patch (+60/-39) plugins/decor/decor.xml.in (+1/-1) plugins/decor/src/compiz-decorator (+67/-67) |
||||
To merge this branch: | bzr merge lp:~bryanfritt/compiz/compiz-decorator_script-edit_1192376 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
MC Return | Pending | ||
Daniel van Vugt | Pending | ||
Sam Spilsbury | Pending | ||
Review via email: mp+176814@code.launchpad.net |
This proposal supersedes a proposal from 2013-06-24.
Commit message
Changed 'compiz-decorator', and 'decor.xml.in':
fixed missing '/'
fixed 'kde-plasma' detection
changed comments
added some quoting
moved some code around
(LP: #1192376)
Description of the change
compiz-decorator changes: 'default path', 'which', 'plasma-
decor.xml.in changes: default command for decorator
Describe what changes your branch introduces,
Changes to compiz-decorator and decor.xml.in that get it to choose and run a decorator better.
"what bugs it fixes,"
Fallback was messed up for KDE,
'/' neaded in paths,
better description of what's it's doing, if it can't find a perfect decorator match,
'--replace &' was missing
"or what features it implements."
If COMPIZ_BIN_PATH isn't set, it'll detect what directory it's in, and set COMPIZ_BIN_PATH to it,
detects location of METACITY,
allows spaces in paths of HOME, XDG_CONFIG_DIRS, XDG_CONFIG_HOME, etc...
"Ideally include rationale and how to test."
Kill the the decorators, and then run this script in COMPIZ_BIN_PATH, or in the paths with the decorators,
Are the correct decorators for KDE and GNOME run?
MC Return (mc-return) wrote : Posted in a previous version of this proposal | # |
MC Return (mc-return) wrote : Posted in a previous version of this proposal | # |
BryanFRitt (bryanfritt) wrote : Posted in a previous version of this proposal | # |
> If this MP is ready, please set it's status from "Work in progress" to "Needs
> review" and do not forget to add a main commit message to it via the "Set
> commit message"- (green "+") button above...
>
> Another note: If you "bzr merge lp:compiz" and then commit, you do not have to
> copy the whole commit message, it is enough to say "Merged latest lp:compiz"
> or "Merged lp:compiz" in the commit message then...
What does 'MP' stand for?
I don't see a "work in progress"/"Needs review". Where is it? The closest thing I see is "Request a review".
BryanFRitt (bryanfritt) wrote : Posted in a previous version of this proposal | # |
For the 'commit message(s)' is there a specific writing style that's preferred?
MC Return (mc-return) wrote : Posted in a previous version of this proposal | # |
MP == Merge Proposal
Search "Work in progress" on this page via browser if you cannot find it ;)
Well, there is no hard rule regarding the commit message, but:
I always start uppercase and end with punctuation, but your commit message is okay that way as well...
It would be good to add "(LP: #1192376)" at the bottom to have the link to the bug directly in the commit message.
I hope Sam can take a look at this also soon, as I am no expert regarding this script, but generally this looks good already and I cannot find anything that is obviously wrong.
MC Return (mc-return) wrote : Posted in a previous version of this proposal | # |
LGTM. +1.
Thanks 4 all the work you invested into this. Top job !
Hope to see more MPs from you in the future. Good to know that KDE users still care about Compiz !
Approve from me, but waiting for Sam with the final green light, but from what I am seeing everything looks good. :)
BryanFRitt (bryanfritt) wrote : Posted in a previous version of this proposal | # |
I've edited/started/done a start-compiz and a start-ccsm script, they export LD_LIBRARY_PATH, and PYTHONPATH, etc...
How can I get them into Compiz?
They would be new files that aren't in there currently, so I have to find and mess with an install script and add them in somehow?
(and/or perhaps Compiz could be made so that these scripts would be optional)
Guess I start a new branch/bug report?
-----
#!/bin/sh
# start-compiz
# Assumes script is in a direct sibdirectory of COMPIZ_PREFIX, like "${COMPIZ_
COMPIZ_
# If the above isn't correct, it can be commented it out, and the line below edited
#COMPIZ_PREFIX=
#VERBOSE="no"
VERBOSE="yes"
# Echos the arguments if verbose
verbose()
{
if [ "${VERBOSE}" = "yes" ]; then
echo "$@"
fi
}
verbose "COMPIZ_
#export PKG_CONFIG_
export LD_LIBRARY_
verbose "LD_LIBRARY_
if [ -n `which "python2.7"` ]; then
PYTHONVERSION=
else
if [ -n `which "python2.6"` ]; then
PYTHONVERSION
else
echo "neither python2.7 nor python2.6 where found"
fi
fi
verbose "PYTHONVERSION=
if [ -n "${PYTHONVERSION}" ]; then
#unset PYTHONPATH
export PYTHONPATH=
verbose "PYTHONPATH=
fi
# gets rid of plama panels, etc... probably not good
#kquitapp plasma-desktop
verbose "${COMPIZ_
"${COMPIZ_
-----
#!/bin/sh
# start-ccsm
# Kill any running instance of ccsm
#pkill -x ccsm
# Assumes script is in a direct subdirectory of $COMPIZ_PREFIX, like "${COMPIZ_
COMPIZ_
# If the above isn't correct, it can be commented it out, and the line below edited
#COMPIZ_PREFIX=
VERBOSE="no"
#VERBOSE="yes"
# Echos the arguments if verbose
verbose()
{
if [ "${VERBOSE}" = "yes" ]; then
echo "$@"
fi
}
verbose "COMPIZ_
export PKG_CONFIG_
verbose "PKG_CONFIG_
export LD_LIBRARY_
verbose "LD_LIBRARY_
# with this way somehow in either bash or sh, with this, python editor will start, not the intended result!
#if $(`which "python2.7"` &> /dev/null); then
# export PYTHONPATH=
#fi
#if $(which python2.6 &> /dev/null); then
# export PYTHONPATH=
BryanFRitt (bryanfritt) wrote : Posted in a previous version of this proposal | # |
> LGTM. +1.
> Thanks 4 all the work you invested into this. Top job !
> Hope to see more MPs from you in the future. Good to know that KDE users still
> care about Compiz !
>
> Approve from me, but waiting for Sam with the final green light, but from what
> I am seeing everything looks good. :)
Couldn't have done it without your help on bzr, etc...! Thanks!
MC Return (mc-return) wrote : Posted in a previous version of this proposal | # |
>
> I've edited/started/done a start-compiz and a start-ccsm script, they export
> LD_LIBRARY_PATH, and PYTHONPATH, etc...
You seem to be the script master ;)
Great -> useful scripts are very welcome. +1 for engaging this.
> How can I get them into Compiz?
> They would be new files that aren't in there currently, so I have to find and
> mess with an install script and add them in somehow?
> (and/or perhaps Compiz could be made so that these scripts would be optional)
>
You can add directories and files with "bzr add filename".
We already have a scripts/ directory in the Compiz source, so simply add your script(s) with "bzr add scripts/
> Guess I start a new branch/bug report?
>
Yes, please. Unrelated issues should not be mixed. Also it helps review to have everything separated.
Please also take care of typos and try to commit best-quality stuff only, make no compromises there ;)
I guess GPL v2 licensing of your code is okay for you ?
MC Return (mc-return) wrote : Posted in a previous version of this proposal | # |
>
> Couldn't have done it without your help on bzr, etc...! Thanks!
No problem. We all want a better, cooler and greater Compiz don't we ? ;)
MC Return (mc-return) wrote : Posted in a previous version of this proposal | # |
Another note:
Do not worry if a requested review can take a while. We are all volunteers here and time is precious these days... ;)
So do not wait for review of your old MP, if you have further MPs -> bring them on.
Everything gets reviewed sooner or later ;)
BryanFRitt (bryanfritt) wrote : Posted in a previous version of this proposal | # |
Is there a way/how to 'undo' a merge commit like 3752? Like if last merge were to prevent this branch from compiling, or something else?
I'm guessing doing an unnecessary bzr merge or a bzr merge without a reason is a bad thing?
I supose it would affect those downloading the branch, if they chose to try to compile it, etc..., but would it even matter to those evaluating the merge proposal in my case since it's the last thing done, and any of the files I changed, weren't changed by others?
BryanFRitt (bryanfritt) wrote : Posted in a previous version of this proposal | # |
The latest Compiz is giving me an error during 'make'.
...
compiz/
cc1plus: error: unrecognized command line option "-Wno-unused-
cc1plus: error: unrecognized command line option "-Wno-unused-
cc1plus: error: unrecognized command line option "-Wno-unused-
cc1plus: all warnings being treated as errors
make[2]: *** [kde/window-
make[1]: *** [kde/window-
make: *** [all] Error 2
Sam Spilsbury (smspillaz) wrote : Posted in a previous version of this proposal | # |
You can disable KDE4 builds by doing -DBUILD_KDE4=OFF
On Fri, Jun 28, 2013 at 11:18 PM, BryanFRitt <email address hidden> wrote:
> The latest Compiz is giving me an error during 'make'.
> ...
> compiz/
> cc1plus: error: unrecognized command line option "-Wno-unused-
> cc1plus: error: unrecognized command line option "-Wno-unused-
> cc1plus: error: unrecognized command line option "-Wno-unused-
> cc1plus: all warnings being treated as errors
> make[2]: *** [kde/window-
> make[1]: *** [kde/window-
> make: *** [all] Error 2
> --
> https:/
> You are requested to review the proposed merge of lp:~bryanfritt/compiz/compiz-decorator_script-edit_1192376 into lp:compiz.
--
Sam Spilsbury
MC Return (mc-return) wrote : Posted in a previous version of this proposal | # |
> Is there a way/how to 'undo' a merge commit like 3752? Like if last merge were
> to prevent this branch from compiling, or something else?
>
"bzr revert" will revert uncommitted changes, use "bzr revert --help" for detailed
and verbose instructions on how to remove a commit.
> I'm guessing doing an unnecessary bzr merge or a bzr merge without a reason is
> a bad thing?
You just sync your source with trunk, nothing bad about doing this. It happens
anyway, when your branch lands. But doing it before it lands will give you the
opportunity to fix merge problems before they occur.
> I supose it would affect those downloading the branch, if they chose to try to
> compile it, etc..., but would it even matter to those evaluating the merge
> proposal in my case since it's the last thing done, and any of the files I
> changed, weren't changed by others?
I'm sorry, I do not understand what you mean by this...
MC Return (mc-return) wrote : Posted in a previous version of this proposal | # |
I recommend to use cmake-gui to configure build options as it gives a nice visual representation of all the options.
BryanFRitt (bryanfritt) wrote : Posted in a previous version of this proposal | # |
> You can disable KDE4 builds by doing -DBUILD_KDE4=OFF
>
> On Fri, Jun 28, 2013 at 11:18 PM, BryanFRitt <email address hidden> wrote:
> > The latest Compiz is giving me an error during 'make'.
> > ...
> > compiz/
> kde4/window.
> > cc1plus: error: unrecognized command line option "-Wno-unused-
> [-Werror]
> > cc1plus: error: unrecognized command line option "-Wno-unused-
> [-Werror]
> > cc1plus: error: unrecognized command line option "-Wno-unused-
> [-Werror]
> > cc1plus: all warnings being treated as errors
> > make[2]: *** [kde/window-
> decorator.
> > make[1]: *** [kde/window-
> decorator.dir/all] Error 2
> > make: *** [all] Error 2
> > --
> > https:/
> edit_1192376/
> > You are requested to review the proposed merge of lp:~bryanfritt/compiz
> /compiz-
>
>
>
> --
> Sam Spilsbury
It compiles with -DBUILD_KDE4=OFF. I'm using KDE and the 'kde4-window-
BryanFRitt (bryanfritt) wrote : Posted in a previous version of this proposal | # |
> > I supose it would affect those downloading the branch, if they chose to try
> to
> > compile it, etc..., but would it even matter to those evaluating the merge
> > proposal in my case since it's the last thing done, and any of the files I
> > changed, weren't changed by others?
>
> I'm sorry, I do not understand what you mean by this...
I guess I was trying to say the script working and Compiz compiling are fairly independent. Even if the script was replaced with complete garbage, it wouldn't affect rather or not Compiz compiles. But to those who would get the branch by itself `bzr branch lp:~bryanfritt/compiz/compiz-decorator_script-edit_1192376` and try to compile that would be affected by rather or not this Compiz branch compiles, or not.
Sam Spilsbury (smspillaz) wrote : Posted in a previous version of this proposal | # |
61 +# Set COMPIZ_BIN_PATH if need be
62 +if [ -z "${COMPIZ_
63 + #COMPIZ_
64 + # ?set default to the bin path is in the same path as this file, or to do above?
65 + COMPIZ_
66 + #echo "${COMPIZ_
67 +fi
This is fine, although the method is a bit strange. Is there anything wrong with:
if [ -z "${COMPIZ_
# Set default binary path to the same path as this file
COMPIZ_
fi
There's lots of commented out code in the review version.
115 -if [ -n "$DECORATOR" ]; then
116 - verbose "Starting ${DECORATOR}\n"
117 - exec ${COMPIZ_
118 -else
119 - verbose "Found no decorator to start\n"
120 - exec $FALLBACKWM $FALLBACKWM_OPTIONS
121 +if [ -z "${XDG_
122 + test -f "/etc/xdg/
123 +else
124 + test -f "${XDG_
125 +fi
These changes are inconsistent with the coding style. Use 8-wide tabs for two indents and 4-spaces for one indent, or, if the language doesn't support it, use 4-spaces for indents.
159 + # If there isn't a compiz decorator available, try not leave users without decoration
160 + if [ "${DESKTOP_
161 + FALLBACKWM=`which kwin` # typically "/usr/bin/kwin"
162 + else
163 + FALLBACKWM=`which metacity` # typically "/usr/bin/metacity"
164 + fi
165 + FALLBACKWM_
166 + verbose "Couldn't find a decorator to start, running fallback"
167 + verbose "exec ${FALLBACKWM} ${FALLBACKWM_
168 + exec "${FALLBACKWM}" "${FALLBACKWM_
I'm not so sure how I feel about starting another _window manager_ just because a decorator wasn't found, but this code was here before so I don't mind too much.
BryanFRitt (bryanfritt) wrote : Posted in a previous version of this proposal | # |
COMPIZ_
The first always gives the full path; The second changes based on how it's called. Put it in a script and it'll change based on how the script is called. Like is it called via a relative path, or via a full path? I suppose for this script this line could be done either way, although I haven't tested the script with the second.
you can try it out...
put this in a file and run different ways
# a quickly made example...
# with this code in /opt/CompizBZR/
THIS_PATH_1=$(cd $(dirname "$0"); pwd)
echo $THIS_PATH_1
THIS_PATH_
echo $THIS_PATH_2
# results:
# called from /opt/CompizBZR/ with /opt/CompizBZR/
# /opt/CompizBZR/bin
# /opt/CompizBZR/bin
# also called from /opt/CompizBZR/ with ./bin/pathTests.sh
# /opt/CompizBZR/bin
# ./bin
# called from ~/ with ../../opt/
# /opt/CompizBZR/bin
# ../../opt/
# called from /opt/CompizBZR/bin/ with ./pathTests.sh
# ./pathTests.sh
# /opt/CompizBZR/bin
"There's lots of commented out code in the review version."
This was my first time messing with bzr, and Compiz submissions. I'm guessing I hit submit for review too soon?, and guessing it doesn't update which code to review? and so you might reviewed an earlier version?
(And I should resubmit again after changing tabs to 4 spaces?)
The one I should have submitted should have been done after 3751 `chmod +x`, the last thing I've done with this code. I hadn't thought of any changes to do with it since then, and it's working for me, so I guess I'm done with this set of edits, except for now messing with what you said, the using four spaces in place of a tab.
"These changes are inconsistent with the coding style. Use 8-wide tabs for two indents and 4-spaces for one indent, or, if the language doesn't support it, use 4-spaces for indents."
I didn't know about the coding style. Where can I read it?
Should be an easy change. No problem. It'll actually be easier to do copy paste code type testing on to the shell with spaces instead of tab. (tab characters cause 'tab completions' to activate during copy paste, which makes things look weird...)
"I'm not so sure how I feel about starting another _window manager_ just because a decorator wasn't found, but this code was here before so I don't mind too much."
I thought the same thing, and left it in there as a 'just in case', and 'it must have been added for a reason'..., If this code causes problems, it can be taken out without too much trouble.
Sam Spilsbury (smspillaz) wrote : Posted in a previous version of this proposal | # |
> COMPIZ_
> The first always gives the full path; The second changes based on how it's
> called. Put it in a script and it'll change based on how the script is called.
> Like is it called via a relative path, or via a full path? I suppose for this
> script this line could be done either way, although I haven't tested the
> script with the second.
>
> you can try it out...
> put this in a file and run different ways
>
> # a quickly made example...
> # with this code in /opt/CompizBZR/
>
> THIS_PATH_1=$(cd $(dirname "$0"); pwd)
> echo $THIS_PATH_1
>
> THIS_PATH_
> echo $THIS_PATH_2
>
> # results:
> # called from /opt/CompizBZR/ with /opt/CompizBZR/
> # /opt/CompizBZR/bin
> # /opt/CompizBZR/bin
>
> # also called from /opt/CompizBZR/ with ./bin/pathTests.sh
> # /opt/CompizBZR/bin
> # ./bin
>
> # called from ~/ with ../../opt/
> # /opt/CompizBZR/bin
> # ../../opt/
>
> # called from /opt/CompizBZR/bin/ with ./pathTests.sh
> # ./pathTests.sh
> # /opt/CompizBZR/bin
Ah okay, what I might suggest is readlink -e, eg
readlink -e "$0"
That works regardless of the path, and it also resolves symlinks too.
Changing the pwd is a bit of a kludge and can get messy if the script crashes or gets killed in-between.
>
> "There's lots of commented out code in the review version."
> This was my first time messing with bzr, and Compiz submissions. I'm guessing
> I hit submit for review too soon?, and guessing it doesn't update which code
> to review? and so you might reviewed an earlier version?
> (And I should resubmit again after changing tabs to 4 spaces?)
It will automatically update. Resubmit if you want - it just sends me another email notification that's all :)
>
> The one I should have submitted should have been done after 3751 `chmod +x`,
> the last thing I've done with this code. I hadn't thought of any changes to do
> with it since then, and it's working for me, so I guess I'm done with this set
> of edits, except for now messing with what you said, the using four spaces in
> place of a tab.
>
> "These changes are inconsistent with the coding style. Use 8-wide tabs for two
> indents and 4-spaces for one indent, or, if the language doesn't support it,
> use 4-spaces for indents."
> I didn't know about the coding style. Where can I read it?
> Should be an easy change. No problem. It'll actually be easier to do copy
> paste code type testing on to the shell with spaces instead of tab. (tab
> characters cause 'tab completions' to activate during copy paste, which makes
> things look weird...)
>
Sam Spilsbury (smspillaz) wrote : Posted in a previous version of this proposal | # |
> The one I should have submitted should have been done after 3751 `chmod +x`,
> the last thing I've done with this code. I hadn't thought of any changes to do
> with it since then, and it's working for me, so I guess I'm done with this set
> of edits, except for now messing with what you said, the using four spaces in
> place of a tab.
>
> "These changes are inconsistent with the coding style. Use 8-wide tabs for two
> indents and 4-spaces for one indent, or, if the language doesn't support it,
> use 4-spaces for indents."
> I didn't know about the coding style. Where can I read it?
> Should be an easy change. No problem. It'll actually be easier to do copy
> paste code type testing on to the shell with spaces instead of tab. (tab
> characters cause 'tab completions' to activate during copy paste, which makes
> things look weird...)
>
Oops - I was going to mention. This is on wiki.compiz.org, but that's down again. I'll grab the link as soon as it comes back up. Basically:
1. 8-wide tab for two indents, 4 spaces for the last indent for odd indent levels
2. Space in-between () and identifiers
3. camelCase
4. Braces fall on the same line as the start of the control condition, eg:
if (1)
{
}
BryanFRitt (bryanfritt) wrote : Posted in a previous version of this proposal | # |
"This is on wiki.compiz.org, but that's down again. I'll grab the link as soon as it comes back up."
It's been down for quite a while, a "(it no longer loads, at least for me)" was written "2013-06-30", and it's still down for me.
"These changes are inconsistent with the coding style. Use 8-wide tabs for two indents and 4-spaces for one indent, or, if the language doesn't support it, use 4-spaces for indents."
"1. 8-wide tab for two indents, 4 spaces for the last indent for odd indent levels"
I replaced every indention tab with four spaces. Upon reading again, I'm not sure if that's what you meant. What did you mean by "last indent", and "odd indent levels", etc...?
Sam Spilsbury (smspillaz) wrote : Posted in a previous version of this proposal | # |
So, for example:
1 indent: 4 spaces
2 indents: 1 tab
3 indents: 1 tab, 4 spaces
4 indents: 2 tabs
5 indents: 2 tabs, 4 spaces
etc
Daniel van Vugt (vanvugt) wrote : Posted in a previous version of this proposal | # |
Please re-target to lp:compiz
BryanFRitt (bryanfritt) wrote : Posted in a previous version of this proposal | # |
> So, for example:
>
> 1 indent: 4 spaces
> 2 indents: 1 tab
> 3 indents: 1 tab, 4 spaces
> 4 indents: 2 tabs
> 5 indents: 2 tabs, 4 spaces
>
> etc
Even with a Bash script too? If you try to drag and drop a line of code onto a Bash terminal window, everywhere there's a tab it'll try to do a tab completion(`ls` or whatever it's called). Which looks weird, even though it doesn't affect how the code runs. Spaces instead of tabs would be good for those who want to test parts of the script out this way.
> camelCase
In Bash environment variables, and global variables, are conventionally done in all UPPERCASE
http://
> Please re-target to lp:compiz
What does this mean? / How to?
MC Return (mc-return) wrote : Posted in a previous version of this proposal | # |
> > So, for example:
> >
> > 1 indent: 4 spaces
> > 2 indents: 1 tab
> > 3 indents: 1 tab, 4 spaces
> > 4 indents: 2 tabs
> > 5 indents: 2 tabs, 4 spaces
> >
> > etc
>
> Even with a Bash script too? If you try to drag and drop a line of code onto a
> Bash terminal window, everywhere there's a tab it'll try to do a tab
> completion(`ls` or whatever it's called). Which looks weird, even though it
> doesn't affect how the code runs. Spaces instead of tabs would be good for
> those who want to test parts of the script out this way.
>
I agree that the C++ X11 indentation style is not needed and would look weird in bash scripts.
> > camelCase
> In Bash environment variables, and global variables, are conventionally done
> in all UPPERCASE
> http://
> variable-
>
I agree here also.
> > Please re-target to lp:compiz
>
> What does this mean? / How to?
1. Copy your commit message (as you will need it again).
2. Open your branch: lp:~bryanfritt/compiz/compiz-decorator_script-edit_1192376
3. Propose a new merge into lp:compiz, which is Compiz 0.9.11-dev now (this here is now lp:compiz/0.9.10).
4. Paste your commit message again, when you're done.
BryanFRitt (bryanfritt) wrote : Posted in a previous version of this proposal | # |
1. Copy your commit message (as you will need it again).
2. Open your branch: lp:~bryanfritt/compiz/compiz-decorator_script-edit_1192376
3. Propose a new merge into lp:compiz, which is Compiz 0.9.11-dev now (this here is now lp:compiz/0.9.10).
4. Paste your commit message again, when you're done.
"The proposal to merge lp:~bryanfritt/compiz/compiz-decorator_script-edit_1192376 into lp:compiz has been updated."
"This proposal has been superseded by a proposal from 2013-07-24."
I guess this means someone got to it before me. (that's fine by me)
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:3754
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
BryanFRitt (bryanfritt) wrote : | # |
> FAILED: Continuous integration, rev:3754
> http://
> Executed test runs:
> FAILURE: http://
> FAILURE: http://
> FAILURE: http://
>
> Click here to trigger a rebuild:
> http://
patching file plugins/
Hunk #4 FAILED at 101.
1 out of 4 hunks FAILED -- rejects in file plugins/
Does that mean it didn't like this?
<default>exec ${COMPIZ_
line 104 in decor.xml.in
Which affects
`ccsm` program->'Effects' section->'Window Decoration' plugin->'General' tab->'Command' option->'Reset' button.
I tried something like
<default>exec "${COMPIZ_
and it didn't compile, until I removed the quotes; so I left it unquoted, and it would compile for me.
It probably should be quoted, but that didn't compile for me, until I removed the quotes, but it seams like it still failed for PS Jenkins bot (ps-jenkins)
with
<default>exec "${COMPIZ_
make gives
.../build/
At global scope:
cc1plus: error: unrecognized command line option "-Wno-unused-
cc1plus: error: unrecognized command line option "-Wno-unused-
cc1plus: error: unrecognized command line option "-Wno-unused-
cc1plus: error: unrecognized command line option "-Wno-unused-
cc1plus: error: unrecognized command line option "-Wno-unused-
cc1plus: error: unrecognized command line option "-Wno-unused-
cc1plus: all warnings being treated as errors
make[2]: *** [plugins/
make[1]: *** [plugins/
make: *** [all] Error 2
tried
exec \"${COMPIZ_
makes the default
exec \"${COMPIZ_
tried
exec ""${COMPIZ_
makes the default
exec ""${COMPIZ_
tried
exec '${COMPIZ_
makes the default
exec '${COMPIZ_
ME=Blah
echo '${ME}/hi'
{ME}/hi
echo "${ME}/hi"
Blah/hi
with
<default>exec "${
make gives
not well-formed (invalid token) at line 104, column 21, byte 3100 at /usr/lib/
make[2]: *** [generated/
make[1]: *** [plugins/
make: *** [all] Error 2
...
How can this be made to make the default
exec "${COMPIZ_
and to be able to compile for me, and PS Jenkins bot (ps-jenkins) ?
Sam Spilsbury (smspillaz) wrote : | # |
You need to update the distro patch:
sudo apt-get install quilt;
export QUILT_PATCHES=
quilt push -fa
*edit the file where hunks from the patch did not apply such that the patch
is applied "by hand"*
quilt refresh
quilt pop -fa
bzr commit
On Wed, Jul 24, 2013 at 10:53 PM, BryanFRitt <email address hidden> wrote:
> > FAILED: Continuous integration, rev:3754
> > http://
> > Executed test runs:
> > FAILURE:
> http://
> > FAILURE:
> http://
> > FAILURE:
> http://
> >
> > Click here to trigger a rebuild:
> > http://
>
> patching file plugins/
> Hunk #4 FAILED at 101.
> 1 out of 4 hunks FAILED -- rejects in file plugins/
>
> Does that mean it didn't like this?
> <default>exec ${COMPIZ_
> line 104 in decor.xml.in
>
> Which affects
> `ccsm` program->'Effects' section->'Window Decoration' plugin->'General'
> tab->'Command' option->'Reset' button.
>
> I tried something like
> <default>exec "${COMPIZ_
> and it didn't compile, until I removed the quotes; so I left it unquoted,
> and it would compile for me.
>
> It probably should be quoted, but that didn't compile for me, until I
> removed the quotes, but it seams like it still failed for PS Jenkins bot
> (ps-jenkins)
>
> with
> <default>exec "${COMPIZ_
> make gives
> .../build/
> At global scope:
> cc1plus: error: unrecognized command line option
> "-Wno-unused-
> cc1plus: error: unrecognized command line option
> "-Wno-unused-
> cc1plus: error: unrecognized command line option
> "-Wno-unused-
> cc1plus: error: unrecognized command line option
> "-Wno-unused-
> cc1plus: error: unrecognized command line option
> "-Wno-unused-
> cc1plus: error: unrecognized command line option
> "-Wno-unused-
> cc1plus: all warnings being treated as errors
> make[2]: ***
> [plugins/
> Error 1
> make[1]: *** [plugins/
> make: *** [all] Error 2
>
> tried
> exec \"${COMPIZ_
> makes the default
> exec \"${COMPIZ_
>
> tried
> exec ""${COMPIZ_
> makes the default
> exec ""${COMPIZ_
>
> tried
> exec '${COMPIZ_
> makes the default
> exec '${COMPIZ_
>
> ME=Blah
> echo '${ME}/hi'
> {ME}/hi
>
> echo "${ME}/hi"
> Blah/hi
>
> with
> <default>exec "${
> make gives
> not well-formed (invalid token) at line 104, column 21, byte 3100 at
> /usr/lib/
> make[2]: *** [gener...
- 3755. By BryanFRitt
-
https:/
/code.launchpad .net/~bryanfrit t/compiz/ compiz- decorator_ script- edit_1192376/ +merge/ 176814/ comments/ 397598
"update the distro patch"
BryanFRitt (bryanfritt) wrote : | # |
> You need to update the distro patch:
>
> sudo apt-get install quilt;
> export QUILT_PATCHES=
> quilt push -fa
> *edit the file where hunks from the patch did not apply such that the patch
> is applied "by hand"*
> quilt refresh
> quilt pop -fa
> bzr commit
patching file plugins/
Hunk #4 FAILED at 101.
1 out of 4 hunks FAILED -- saving rejects to file plugins/
<default>
vs.
<default>exec \"${COMPIZ_
vs.
<default>exec ${COMPIZ_
and
exec "${COMPIZ_
Question:
Is COMPIZ_BIN_PATH set up automatically for this, or does the user have to set it up?
`ccsm` program->'Effects' section->'Window Decoration' plugin->'General' tab->'Command' option->...
Even though this can't put quote around the path (so the path could have a space), it's still better than <default>exec \"${COMPIZ_
PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:3755
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
Unmerged revisions
- 3755. By BryanFRitt
-
https:/
/code.launchpad .net/~bryanfrit t/compiz/ compiz- decorator_ script- edit_1192376/ +merge/ 176814/ comments/ 397598
"update the distro patch" - 3754. By BryanFRitt
-
improved comments
- 3753. By BryanFRitt
-
bzr merge, Merged lp:compiz
- 3752. By BryanFRitt
-
replaced tabs with 4 spaces
and replaced
COMPIZ_BIN_PATH= $(cd $(dirname "$0"); pwd)
with
COMPIZ_BIN_PATH= "$(dirname $(readlink -e "$0"))"
https://code.launchpad .net/~bryanfrit t/compiz/ compiz- decorator_ script- edit_1192376/ +merge/ 171005/ comments/ 391959 - 3751. By BryanFRitt
-
chmod +x
- 3750. By BryanFRitt
-
removed some '\n'
- 3749. By BryanFRitt
-
Changed an 'echo' to a 'verbose'.
- 3748. By BryanFRitt
-
replaced a printf "$*" with a echo "$@" ...
- 3747. By BryanFRitt
-
moved stuff around
- 3746. By BryanFRitt
-
expanded quoting range
Preview Diff
1 | === modified file 'debian/patches/ubuntu-config.patch' | |||
2 | --- debian/patches/ubuntu-config.patch 2013-07-23 23:30:23 +0000 | |||
3 | +++ debian/patches/ubuntu-config.patch 2013-07-25 07:29:27 +0000 | |||
4 | @@ -1,6 +1,8 @@ | |||
5 | 1 | === modified file 'metadata/core.xml.in' | 1 | === modified file 'metadata/core.xml.in' |
8 | 2 | --- a/metadata/core.xml.in | 2 | Index: compiz-decorator_script-edit_1192376/metadata/core.xml.in |
9 | 3 | +++ b/metadata/core.xml.in | 3 | =================================================================== |
10 | 4 | --- compiz-decorator_script-edit_1192376.orig/metadata/core.xml.in 2013-06-27 20:22:24.000000000 -0400 | ||
11 | 5 | +++ compiz-decorator_script-edit_1192376/metadata/core.xml.in 2013-07-25 02:20:57.000000000 -0400 | ||
12 | 4 | @@ -139,7 +139,7 @@ | 6 | @@ -139,7 +139,7 @@ |
13 | 5 | <option name="focus_prevention_match" type="match"> | 7 | <option name="focus_prevention_match" type="match"> |
14 | 6 | <_short>Focus Prevention Windows</_short> | 8 | <_short>Focus Prevention Windows</_short> |
15 | @@ -63,8 +65,10 @@ | |||
16 | 63 | <min>1</min> | 65 | <min>1</min> |
17 | 64 | <max>32</max> | 66 | <max>32</max> |
18 | 65 | </option> | 67 | </option> |
21 | 66 | --- a/plugins/animation/animation.xml.in | 68 | Index: compiz-decorator_script-edit_1192376/plugins/animation/animation.xml.in |
22 | 67 | +++ b/plugins/animation/animation.xml.in | 69 | =================================================================== |
23 | 70 | --- compiz-decorator_script-edit_1192376.orig/plugins/animation/animation.xml.in 2013-06-27 20:22:24.000000000 -0400 | ||
24 | 71 | +++ compiz-decorator_script-edit_1192376/plugins/animation/animation.xml.in 2013-07-25 02:20:57.000000000 -0400 | ||
25 | 68 | @@ -36,7 +36,7 @@ | 72 | @@ -36,7 +36,7 @@ |
26 | 69 | <extensible/> | 73 | <extensible/> |
27 | 70 | <sort start="2"/> | 74 | <sort start="2"/> |
28 | @@ -203,8 +207,10 @@ | |||
29 | 203 | <min>0</min> | 207 | <min>0</min> |
30 | 204 | <max>1</max> | 208 | <max>1</max> |
31 | 205 | <precision>0.01</precision> | 209 | <precision>0.01</precision> |
34 | 206 | --- a/plugins/decor/decor.xml.in | 210 | Index: compiz-decorator_script-edit_1192376/plugins/decor/decor.xml.in |
35 | 207 | +++ b/plugins/decor/decor.xml.in | 211 | =================================================================== |
36 | 212 | --- compiz-decorator_script-edit_1192376.orig/plugins/decor/decor.xml.in 2013-06-27 20:22:24.000000000 -0400 | ||
37 | 213 | +++ compiz-decorator_script-edit_1192376/plugins/decor/decor.xml.in 2013-07-25 02:20:57.000000000 -0400 | ||
38 | 208 | @@ -31,7 +31,7 @@ | 214 | @@ -31,7 +31,7 @@ |
39 | 209 | <option name="active_shadow_opacity" type="float"> | 215 | <option name="active_shadow_opacity" type="float"> |
40 | 210 | <_short>Shadow Opacity</_short> | 216 | <_short>Shadow Opacity</_short> |
41 | @@ -232,17 +238,10 @@ | |||
42 | 232 | <min>0.01</min> | 238 | <min>0.01</min> |
43 | 233 | <max>6.0</max> | 239 | <max>6.0</max> |
44 | 234 | <precision>0.01</precision> | 240 | <precision>0.01</precision> |
56 | 235 | @@ -101,7 +101,7 @@ | 241 | Index: compiz-decorator_script-edit_1192376/plugins/fade/fade.xml.in |
57 | 236 | <option name="command" type="string"> | 242 | =================================================================== |
58 | 237 | <_short>Command</_short> | 243 | --- compiz-decorator_script-edit_1192376.orig/plugins/fade/fade.xml.in 2013-06-27 20:22:24.000000000 -0400 |
59 | 238 | <_long>Decorator command line that is executed if no decorator is already running.</_long> | 244 | +++ compiz-decorator_script-edit_1192376/plugins/fade/fade.xml.in 2013-07-25 02:20:57.000000000 -0400 |
49 | 239 | - <default>exec \"${COMPIZ_BIN_PATH}compiz-decorator\"</default> | ||
50 | 240 | + <default>/usr/bin/gtk-window-decorator</default> | ||
51 | 241 | </option> | ||
52 | 242 | <option name="mipmap" type="bool"> | ||
53 | 243 | <_short>Mipmap</_short> | ||
54 | 244 | --- a/plugins/fade/fade.xml.in | ||
55 | 245 | +++ b/plugins/fade/fade.xml.in | ||
60 | 246 | @@ -10,6 +10,7 @@ | 245 | @@ -10,6 +10,7 @@ |
61 | 247 | </requirement> | 246 | </requirement> |
62 | 248 | <relation type="after"> | 247 | <relation type="after"> |
63 | @@ -260,8 +259,10 @@ | |||
64 | 260 | </option> | 259 | </option> |
65 | 261 | <option name="visual_bell" type="bell"> | 260 | <option name="visual_bell" type="bell"> |
66 | 262 | <_short>Visual Bell</_short> | 261 | <_short>Visual Bell</_short> |
69 | 263 | --- a/plugins/gnomecompat/gnomecompat.xml.in | 262 | Index: compiz-decorator_script-edit_1192376/plugins/gnomecompat/gnomecompat.xml.in |
70 | 264 | +++ b/plugins/gnomecompat/gnomecompat.xml.in | 263 | =================================================================== |
71 | 264 | --- compiz-decorator_script-edit_1192376.orig/plugins/gnomecompat/gnomecompat.xml.in 2013-06-27 20:22:24.000000000 -0400 | ||
72 | 265 | +++ compiz-decorator_script-edit_1192376/plugins/gnomecompat/gnomecompat.xml.in 2013-07-25 02:20:57.000000000 -0400 | ||
73 | 265 | @@ -53,6 +53,7 @@ | 266 | @@ -53,6 +53,7 @@ |
74 | 266 | <option name="run_command_terminal_key" type="key"> | 267 | <option name="run_command_terminal_key" type="key"> |
75 | 267 | <_short>Open a terminal</_short> | 268 | <_short>Open a terminal</_short> |
76 | @@ -270,8 +271,10 @@ | |||
77 | 270 | </option> | 271 | </option> |
78 | 271 | </group> | 272 | </group> |
79 | 272 | </options> | 273 | </options> |
82 | 273 | --- a/plugins/place/place.xml.in | 274 | Index: compiz-decorator_script-edit_1192376/plugins/place/place.xml.in |
83 | 274 | +++ b/plugins/place/place.xml.in | 275 | =================================================================== |
84 | 276 | --- compiz-decorator_script-edit_1192376.orig/plugins/place/place.xml.in 2013-06-27 20:22:24.000000000 -0400 | ||
85 | 277 | +++ compiz-decorator_script-edit_1192376/plugins/place/place.xml.in 2013-07-25 02:20:58.000000000 -0400 | ||
86 | 275 | @@ -20,8 +20,8 @@ | 278 | @@ -20,8 +20,8 @@ |
87 | 276 | <option name="mode" type="int"> | 279 | <option name="mode" type="int"> |
88 | 277 | <_short>Placement Mode</_short> | 280 | <_short>Placement Mode</_short> |
89 | @@ -283,8 +286,10 @@ | |||
90 | 283 | <max>5</max> | 286 | <max>5</max> |
91 | 284 | <desc> | 287 | <desc> |
92 | 285 | <value>0</value> | 288 | <value>0</value> |
95 | 286 | --- a/plugins/resize/resize.xml.in | 289 | Index: compiz-decorator_script-edit_1192376/plugins/resize/resize.xml.in |
96 | 287 | +++ b/plugins/resize/resize.xml.in | 290 | =================================================================== |
97 | 291 | --- compiz-decorator_script-edit_1192376.orig/plugins/resize/resize.xml.in 2013-06-27 20:22:24.000000000 -0400 | ||
98 | 292 | +++ compiz-decorator_script-edit_1192376/plugins/resize/resize.xml.in 2013-07-25 02:20:58.000000000 -0400 | ||
99 | 288 | @@ -28,7 +28,7 @@ | 293 | @@ -28,7 +28,7 @@ |
100 | 289 | <option name="mode" type="int"> | 294 | <option name="mode" type="int"> |
101 | 290 | <_short>Default Resize Mode</_short> | 295 | <_short>Default Resize Mode</_short> |
102 | @@ -329,8 +334,10 @@ | |||
103 | 329 | </default> | 334 | </default> |
104 | 330 | </option> | 335 | </option> |
105 | 331 | <subgroup> | 336 | <subgroup> |
108 | 332 | --- a/plugins/scale/scale.xml.in | 337 | Index: compiz-decorator_script-edit_1192376/plugins/scale/scale.xml.in |
109 | 333 | +++ b/plugins/scale/scale.xml.in | 338 | =================================================================== |
110 | 339 | --- compiz-decorator_script-edit_1192376.orig/plugins/scale/scale.xml.in 2013-06-27 20:22:24.000000000 -0400 | ||
111 | 340 | +++ compiz-decorator_script-edit_1192376/plugins/scale/scale.xml.in 2013-07-25 02:20:58.000000000 -0400 | ||
112 | 334 | @@ -19,14 +19,14 @@ | 341 | @@ -19,14 +19,14 @@ |
113 | 335 | <option name="spacing" type="int"> | 342 | <option name="spacing" type="int"> |
114 | 336 | <_short>Spacing</_short> | 343 | <_short>Spacing</_short> |
115 | @@ -419,8 +426,10 @@ | |||
116 | 419 | </option> | 426 | </option> |
117 | 420 | </group> | 427 | </group> |
118 | 421 | </options> | 428 | </options> |
121 | 422 | --- a/plugins/staticswitcher/staticswitcher.xml.in | 429 | Index: compiz-decorator_script-edit_1192376/plugins/staticswitcher/staticswitcher.xml.in |
122 | 423 | +++ b/plugins/staticswitcher/staticswitcher.xml.in | 430 | =================================================================== |
123 | 431 | --- compiz-decorator_script-edit_1192376.orig/plugins/staticswitcher/staticswitcher.xml.in 2013-06-27 20:22:24.000000000 -0400 | ||
124 | 432 | +++ compiz-decorator_script-edit_1192376/plugins/staticswitcher/staticswitcher.xml.in 2013-07-25 02:20:58.000000000 -0400 | ||
125 | 424 | @@ -11,7 +11,6 @@ | 433 | @@ -11,7 +11,6 @@ |
126 | 425 | <relation type="after"> | 434 | <relation type="after"> |
127 | 426 | <plugin>composite</plugin> | 435 | <plugin>composite</plugin> |
128 | @@ -497,8 +506,10 @@ | |||
129 | 497 | <desc> | 506 | <desc> |
130 | 498 | <value>0</value> | 507 | <value>0</value> |
131 | 499 | <_name>None</_name> | 508 | <_name>None</_name> |
134 | 500 | --- a/plugins/vpswitch/vpswitch.xml.in | 509 | Index: compiz-decorator_script-edit_1192376/plugins/vpswitch/vpswitch.xml.in |
135 | 501 | +++ b/plugins/vpswitch/vpswitch.xml.in | 510 | =================================================================== |
136 | 511 | --- compiz-decorator_script-edit_1192376.orig/plugins/vpswitch/vpswitch.xml.in 2013-06-27 20:22:24.000000000 -0400 | ||
137 | 512 | +++ compiz-decorator_script-edit_1192376/plugins/vpswitch/vpswitch.xml.in 2013-07-25 02:20:58.000000000 -0400 | ||
138 | 502 | @@ -95,13 +95,11 @@ | 513 | @@ -95,13 +95,11 @@ |
139 | 503 | <option name="next_button" type="button"> | 514 | <option name="next_button" type="button"> |
140 | 504 | <_short>Move Next</_short> | 515 | <_short>Move Next</_short> |
141 | @@ -513,8 +524,10 @@ | |||
142 | 513 | <internal/> | 524 | <internal/> |
143 | 514 | </option> | 525 | </option> |
144 | 515 | <option name="initiate_button" type="button"> | 526 | <option name="initiate_button" type="button"> |
147 | 516 | --- a/plugins/wall/wall.xml.in | 527 | Index: compiz-decorator_script-edit_1192376/plugins/wall/wall.xml.in |
148 | 517 | +++ b/plugins/wall/wall.xml.in | 528 | =================================================================== |
149 | 529 | --- compiz-decorator_script-edit_1192376.orig/plugins/wall/wall.xml.in 2013-06-27 20:22:24.000000000 -0400 | ||
150 | 530 | +++ compiz-decorator_script-edit_1192376/plugins/wall/wall.xml.in 2013-07-25 02:20:58.000000000 -0400 | ||
151 | 518 | @@ -30,12 +30,12 @@ | 531 | @@ -30,12 +30,12 @@ |
152 | 519 | <option name="miniscreen" type="bool"> | 532 | <option name="miniscreen" type="bool"> |
153 | 520 | <_short>Show Live Viewport Previews</_short> | 533 | <_short>Show Live Viewport Previews</_short> |
154 | @@ -662,8 +675,10 @@ | |||
155 | 662 | </option> | 675 | </option> |
156 | 663 | <option name="edgeflip_dnd" type="bool"> | 676 | <option name="edgeflip_dnd" type="bool"> |
157 | 664 | <_short>Edge Flip DnD</_short> | 677 | <_short>Edge Flip DnD</_short> |
160 | 665 | --- a/tests/system/xorg-gtest/tests/compiz_xorg_gtest_ewmh.cpp | 678 | Index: compiz-decorator_script-edit_1192376/tests/system/xorg-gtest/tests/compiz_xorg_gtest_ewmh.cpp |
161 | 666 | +++ b/tests/system/xorg-gtest/tests/compiz_xorg_gtest_ewmh.cpp | 679 | =================================================================== |
162 | 680 | --- compiz-decorator_script-edit_1192376.orig/tests/system/xorg-gtest/tests/compiz_xorg_gtest_ewmh.cpp 2013-06-27 20:22:24.000000000 -0400 | ||
163 | 681 | +++ compiz-decorator_script-edit_1192376/tests/system/xorg-gtest/tests/compiz_xorg_gtest_ewmh.cpp 2013-07-25 02:20:58.000000000 -0400 | ||
164 | 667 | @@ -46,7 +46,7 @@ | 682 | @@ -46,7 +46,7 @@ |
165 | 668 | 683 | ||
166 | 669 | namespace | 684 | namespace |
167 | @@ -673,8 +688,10 @@ | |||
168 | 673 | unsigned int DEFAULT_VIEWPORT_HEIGHT = 1; | 688 | unsigned int DEFAULT_VIEWPORT_HEIGHT = 1; |
169 | 674 | 689 | ||
170 | 675 | bool Advance (Display *d, bool r) | 690 | bool Advance (Display *d, bool r) |
173 | 676 | --- a/plugins/grid/grid.xml.in | 691 | Index: compiz-decorator_script-edit_1192376/plugins/grid/grid.xml.in |
174 | 677 | +++ b/plugins/grid/grid.xml.in | 692 | =================================================================== |
175 | 693 | --- compiz-decorator_script-edit_1192376.orig/plugins/grid/grid.xml.in 2013-06-27 20:22:24.000000000 -0400 | ||
176 | 694 | +++ compiz-decorator_script-edit_1192376/plugins/grid/grid.xml.in 2013-07-25 02:20:58.000000000 -0400 | ||
177 | 678 | @@ -23,17 +23,16 @@ | 695 | @@ -23,17 +23,16 @@ |
178 | 679 | <option name="put_center_key" type="key"> | 696 | <option name="put_center_key" type="key"> |
179 | 680 | <_short>Put Center Key</_short> | 697 | <_short>Put Center Key</_short> |
180 | @@ -709,8 +726,10 @@ | |||
181 | 709 | </option> | 726 | </option> |
182 | 710 | </group> | 727 | </group> |
183 | 711 | <group> | 728 | <group> |
186 | 712 | --- a/plugins/ezoom/ezoom.xml.in | 729 | Index: compiz-decorator_script-edit_1192376/plugins/ezoom/ezoom.xml.in |
187 | 713 | +++ b/plugins/ezoom/ezoom.xml.in | 730 | =================================================================== |
188 | 731 | --- compiz-decorator_script-edit_1192376.orig/plugins/ezoom/ezoom.xml.in 2013-06-27 20:22:24.000000000 -0400 | ||
189 | 732 | +++ compiz-decorator_script-edit_1192376/plugins/ezoom/ezoom.xml.in 2013-07-25 02:20:58.000000000 -0400 | ||
190 | 714 | @@ -30,7 +30,7 @@ | 733 | @@ -30,7 +30,7 @@ |
191 | 715 | <option type="button" name="zoom_in_button"> | 734 | <option type="button" name="zoom_in_button"> |
192 | 716 | <_short>Zoom In Button</_short> | 735 | <_short>Zoom In Button</_short> |
193 | @@ -747,8 +766,10 @@ | |||
194 | 747 | </option> | 766 | </option> |
195 | 748 | </subgroup> | 767 | </subgroup> |
196 | 749 | </group> | 768 | </group> |
199 | 750 | --- a/plugins/expo/expo.xml.in | 769 | Index: compiz-decorator_script-edit_1192376/plugins/expo/expo.xml.in |
200 | 751 | +++ b/plugins/expo/expo.xml.in | 770 | =================================================================== |
201 | 771 | --- compiz-decorator_script-edit_1192376.orig/plugins/expo/expo.xml.in 2013-07-16 22:31:18.000000000 -0400 | ||
202 | 772 | +++ compiz-decorator_script-edit_1192376/plugins/expo/expo.xml.in 2013-07-25 02:20:58.000000000 -0400 | ||
203 | 752 | @@ -17,6 +17,7 @@ | 773 | @@ -17,6 +17,7 @@ |
204 | 753 | <plugin>wobbly</plugin> | 774 | <plugin>wobbly</plugin> |
205 | 754 | <plugin>animation</plugin> | 775 | <plugin>animation</plugin> |
206 | 755 | 776 | ||
207 | === modified file 'plugins/decor/decor.xml.in' | |||
208 | --- plugins/decor/decor.xml.in 2012-10-15 10:31:51 +0000 | |||
209 | +++ plugins/decor/decor.xml.in 2013-07-25 07:29:27 +0000 | |||
210 | @@ -101,7 +101,7 @@ | |||
211 | 101 | <option name="command" type="string"> | 101 | <option name="command" type="string"> |
212 | 102 | <_short>Command</_short> | 102 | <_short>Command</_short> |
213 | 103 | <_long>Decorator command line that is executed if no decorator is already running.</_long> | 103 | <_long>Decorator command line that is executed if no decorator is already running.</_long> |
215 | 104 | <default>exec \"${COMPIZ_BIN_PATH}compiz-decorator\"</default> | 104 | <default>exec ${COMPIZ_BIN_PATH}/compiz-decorator</default> |
216 | 105 | </option> | 105 | </option> |
217 | 106 | <option name="mipmap" type="bool"> | 106 | <option name="mipmap" type="bool"> |
218 | 107 | <_short>Mipmap</_short> | 107 | <_short>Mipmap</_short> |
219 | 108 | 108 | ||
220 | === modified file 'plugins/decor/src/compiz-decorator' | |||
221 | --- plugins/decor/src/compiz-decorator 2012-06-18 06:22:29 +0000 | |||
222 | +++ plugins/decor/src/compiz-decorator 2013-07-25 07:29:27 +0000 | |||
223 | @@ -1,6 +1,7 @@ | |||
224 | 1 | #!/bin/sh | 1 | #!/bin/sh |
225 | 2 | # | ||
226 | 2 | # Starts Compiz Decorator depending on the DE | 3 | # Starts Compiz Decorator depending on the DE |
228 | 3 | # | 4 | # |
229 | 4 | # Copyright (c) 2007 CyberOrg <cyberorg@cyberorg.info> | 5 | # Copyright (c) 2007 CyberOrg <cyberorg@cyberorg.info> |
230 | 5 | # Based on compiz-manager script by Kristian Lyngstøl <kristian@bohemians.org> | 6 | # Based on compiz-manager script by Kristian Lyngstøl <kristian@bohemians.org> |
231 | 6 | # This program is free software; you can redistribute it and/or modify | 7 | # This program is free software; you can redistribute it and/or modify |
232 | @@ -20,82 +21,81 @@ | |||
233 | 20 | # | 21 | # |
234 | 21 | # Contributions by: crdlb | 22 | # Contributions by: crdlb |
235 | 22 | # Modifications by: Daniel van Vugt <daniel.van.vugt@canonical.com> | 23 | # Modifications by: Daniel van Vugt <daniel.van.vugt@canonical.com> |
247 | 23 | # | 24 | # Additional Changes by: BryanFRitt |
248 | 24 | 25 | ||
249 | 25 | if [ -z "$COMPIZ_BIN_PATH" ]; then | 26 | # Default to gtk-window-decorator or kde4-window-decorator |
239 | 26 | COMPIZ_BIN_PATH="/usr/bin/" | ||
240 | 27 | fi | ||
241 | 28 | KWIN=`which kwin` | ||
242 | 29 | METACITY="/usr/bin/metacity" | ||
243 | 30 | |||
244 | 31 | # | ||
245 | 32 | # Default to gtk/kde4-window-decorator | ||
246 | 33 | # | ||
250 | 34 | USE_EMERALD="no" | 27 | USE_EMERALD="no" |
251 | 35 | DECORATOR="" | 28 | DECORATOR="" |
252 | 36 | |||
253 | 37 | #Do not leave users without decoration if decorator fails | ||
254 | 38 | if [ "$DESKTOP_SESSION" = "kde" ]; then | ||
255 | 39 | FALLBACKWM="${KWIN}" | ||
256 | 40 | else | ||
257 | 41 | FALLBACKWM="${METACITY}" | ||
258 | 42 | fi | ||
259 | 43 | FALLBACKWM_OPTIONS=" --replace" | ||
260 | 44 | |||
261 | 45 | # | ||
262 | 46 | # Set to yes to enable verbose | 29 | # Set to yes to enable verbose |
263 | 47 | # | ||
264 | 48 | VERBOSE="yes" | 30 | VERBOSE="yes" |
267 | 49 | 31 | ||
266 | 50 | # | ||
268 | 51 | # Echos the arguments if verbose | 32 | # Echos the arguments if verbose |
269 | 52 | # | ||
270 | 53 | verbose() | 33 | verbose() |
271 | 54 | { | 34 | { |
274 | 55 | if [ "x$VERBOSE" = "xyes" ]; then | 35 | if [ "${VERBOSE}" = "yes" ]; then |
275 | 56 | printf "$*" | 36 | echo "$@" |
276 | 57 | fi | 37 | fi |
277 | 58 | } | 38 | } |
278 | 59 | 39 | ||
279 | 40 | # Set COMPIZ_BIN_PATH if need be | ||
280 | 41 | if [ -z "${COMPIZ_BIN_PATH}" ]; then | ||
281 | 42 | # two options to setup COMPIZ_BIN_PATH... | ||
282 | 43 | # 1) set COMPIZ_BIN_PATH to a specific directory, like this one: | ||
283 | 44 | #COMPIZ_BIN_PATH="/usr/bin/" | ||
284 | 45 | # xor 2) set COMPIZ_BIN_PATH to the directory that this file is in, like this one: | ||
285 | 46 | COMPIZ_BIN_PATH="$(dirname $(readlink -e "$0"))" | ||
286 | 47 | #verbose "COMPIZ_BIN_PATH=\"${COMPIZ_BIN_PATH}\"" | ||
287 | 48 | fi | ||
288 | 49 | |||
289 | 60 | # Read configuration from XDG paths | 50 | # Read configuration from XDG paths |
329 | 61 | if [ -z "$XDG_CONFIG_DIRS" ]; then | 51 | if [ -z "${XDG_CONFIG_DIRS}" ]; then |
330 | 62 | test -f /etc/xdg/compiz/compiz-manager && . /etc/xdg/compiz/compiz-manager | 52 | test -f "/etc/xdg/compiz/compiz-manager" && . "/etc/xdg/compiz/compiz-manager" |
331 | 63 | else | 53 | else |
332 | 64 | test -f $XDG_CONFIG_DIRS/compiz/compiz-manager && . $XDG_CONFIG_DIRS/compiz/compiz-manager | 54 | test -f "${XDG_CONFIG_DIRS}/compiz/compiz-manager" && . "${XDG_CONFIG_DIRS}/compiz/compiz-manager" |
333 | 65 | fi | 55 | fi |
334 | 66 | 56 | ||
335 | 67 | if [ -z "$XDG_CONFIG_HOME" ]; then | 57 | if [ -z "${XDG_CONFIG_HOME}" ]; then |
336 | 68 | test -f $HOME/.config/compiz/compiz-manager && . $HOME/.config/compiz/compiz-manager | 58 | test -f "${HOME}/.config/compiz/compiz-manager" && . "${HOME}/.config/compiz/compiz-manager" |
337 | 69 | else | 59 | else |
338 | 70 | test -f $XDG_CONFIG_HOME/compiz/compiz-manager && . $XDG_CONFIG_HOME/compiz/compiz-manager | 60 | test -f "${XDG_CONFIG_HOME}/compiz/compiz-manager" && . "${XDG_CONFIG_HOME}/compiz/compiz-manager" |
339 | 71 | fi | 61 | fi |
340 | 72 | 62 | ||
341 | 73 | # start a decorator | 63 | # Pick a decorator |
342 | 74 | if [ -x ${COMPIZ_BIN_PATH}emerald ] && [ "$USE_EMERALD" = "yes" ]; then | 64 | if [ -x "${COMPIZ_BIN_PATH}/emerald" ] && [ "{$USE_EMERALD}" = "yes" ]; then |
343 | 75 | DECORATOR=emerald | 65 | DECORATOR="emerald" |
344 | 76 | elif [ -x ${COMPIZ_BIN_PATH}gtk-window-decorator ] && [ -n "$GNOME_DESKTOP_SESSION_ID" ]; then | 66 | elif [ -x "${COMPIZ_BIN_PATH}/gtk-window-decorator" ] && [ -n "${GNOME_DESKTOP_SESSION_ID}" ]; then |
345 | 77 | DECORATOR=gtk-window-decorator | 67 | DECORATOR="gtk-window-decorator" |
346 | 78 | elif [ -x ${COMPIZ_BIN_PATH}kde4-window-decorator ] && [ x$KDE_SESSION_VERSION = x"4" ]; then | 68 | elif [ -x "${COMPIZ_BIN_PATH}/kde4-window-decorator" ] && [ "${KDE_SESSION_VERSION}" = "4" ]; then |
347 | 79 | DECORATOR=kde4-window-decorator | 69 | DECORATOR="kde4-window-decorator" |
348 | 80 | fi | 70 | fi |
349 | 81 | 71 | ||
350 | 82 | # fall back to any decorator that is installed | 72 | # If a decorator wasn't picked out, fall back to 1st decorator found |
351 | 83 | if [ -z "$DECORATOR" ]; then | 73 | if [ -z "${DECORATOR}" ]; then |
352 | 84 | verbose "Couldn't find a perfect decorator match; trying all decorators\n" | 74 | verbose "Couldn't find a perfect decorator match; using 1st decorator found" |
353 | 85 | if [ -x ${COMPIZ_BIN_PATH}emerald ]; then | 75 | if [ -x "${COMPIZ_BIN_PATH}/emerald" ]; then |
354 | 86 | DECORATOR=emerald | 76 | DECORATOR="emerald" |
355 | 87 | elif [ -x ${COMPIZ_BIN_PATH}gtk-window-decorator ]; then | 77 | elif [ -x "${COMPIZ_BIN_PATH}/gtk-window-decorator" ]; then |
356 | 88 | DECORATOR=gtk-window-decorator | 78 | DECORATOR="gtk-window-decorator" |
357 | 89 | elif [ -x ${COMPIZ_BIN_PATH}kde4-window-decorator ]; then | 79 | elif [ -x "${COMPIZ_BIN_PATH}/kde4-window-decorator" ]; then |
358 | 90 | DECORATOR=kde4-window-decorator | 80 | DECORATOR="kde4-window-decorator" |
359 | 91 | fi | 81 | fi |
360 | 92 | fi | 82 | fi |
361 | 93 | 83 | ||
362 | 94 | if [ -n "$DECORATOR" ]; then | 84 | # Start a decorator |
363 | 95 | verbose "Starting ${DECORATOR}\n" | 85 | if [ -n "${DECORATOR}" ]; then |
364 | 96 | exec ${COMPIZ_BIN_PATH}$DECORATOR "$@" | 86 | verbose "Starting ${DECORATOR}" |
365 | 97 | else | 87 | #verbose "exec \"${COMPIZ_BIN_PATH}/${DECORATOR}\" \"$@\" \"--replace\" &" |
366 | 98 | verbose "Found no decorator to start\n" | 88 | exec "${COMPIZ_BIN_PATH}/${DECORATOR}" "$@" "--replace" & |
367 | 99 | exec $FALLBACKWM $FALLBACKWM_OPTIONS | 89 | else |
368 | 90 | # If there isn't a compiz decorator available, try not leave users without decoration | ||
369 | 91 | if [ "${DESKTOP_SESSION}" = "kde-plasma" ]; then | ||
370 | 92 | FALLBACKWM=`which kwin` # typically "/usr/bin/kwin" | ||
371 | 93 | else | ||
372 | 94 | FALLBACKWM=`which metacity` # typically "/usr/bin/metacity" | ||
373 | 95 | fi | ||
374 | 96 | FALLBACKWM_OPTIONS="--replace" | ||
375 | 97 | verbose "Couldn't find a decorator to start, running fallback" | ||
376 | 98 | verbose "exec ${FALLBACKWM} ${FALLBACKWM_OPTIONS} &" | ||
377 | 99 | exec "${FALLBACKWM}" "${FALLBACKWM_OPTIONS}" & | ||
378 | 100 | fi | 100 | fi |
379 | 101 | 101 |
If this MP is ready, please set it's status from "Work in progress" to "Needs review" and do not forget to add a main commit message to it via the "Set commit message"- (green "+") button above...
Another note: If you "bzr merge lp:compiz" and then commit, you do not have to copy the whole commit message, it is enough to say "Merged latest lp:compiz" or "Merged lp:compiz" in the commit message then...