Merge lp://qastaging/~mc-return/compiz/compiz.merge-plugin-animationjc into lp://qastaging/compiz/0.9.10

Proposed by MC Return
Status: Work in progress
Proposed branch: lp://qastaging/~mc-return/compiz/compiz.merge-plugin-animationjc
Merge into: lp://qastaging/compiz/0.9.10
Diff against target: 1369 lines (+1311/-0)
11 files modified
plugins/animationjc/CMakeLists.txt (+7/-0)
plugins/animationjc/animationjc.xml.in (+154/-0)
plugins/animationjc/compiz-animationjc.pc.in (+12/-0)
plugins/animationjc/include/animationjc/animationjc.h (+343/-0)
plugins/animationjc/src/animationjc.cpp (+145/-0)
plugins/animationjc/src/blackhole.cpp (+79/-0)
plugins/animationjc/src/flicker.cpp (+96/-0)
plugins/animationjc/src/ghost.cpp (+98/-0)
plugins/animationjc/src/popcorn.cpp (+60/-0)
plugins/animationjc/src/private.h (+232/-0)
plugins/animationjc/src/raindrop.cpp (+85/-0)
To merge this branch: bzr merge lp://qastaging/~mc-return/compiz/compiz.merge-plugin-animationjc
Reviewer Review Type Date Requested Status
Daniel van Vugt Pending
Sam Spilsbury Pending
Review via email: mp+156686@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2012-06-17.

Commit message

Added the unsupported, but fully working plug-in "animationjc" converted from git to bzr (including history) to lp:compiz.

Description of the change

Adds the unsupported, but fully working plug-in "animationjc" converted from git to bzr (including history) to Compiz 0.9.8.

To post a comment you must log in.
Revision history for this message
Sam Spilsbury (smspillaz) wrote : Posted in a previous version of this proposal

I feel like animationjc and animationextra should probably be merged into either animation or animationaddon. Two less plugins to maintain. Thoughts?

Revision history for this message
MC Return (mc-return) wrote : Posted in a previous version of this proposal

Black Hole, Flicker, Ghost, Popcorn & Raindrop

Revision history for this message
MC Return (mc-return) wrote : Posted in a previous version of this proposal

Sam, I would fully agree on that.
I have a few other ideas regarding the simplification and optimization of CCSM in general as well: For example hiding plug-ins that depend on other plug-ins and make no sense being shown in CCSM and just confuse the user (like hiding all cube-related plug-ins in CCSM if the cube is disabled for example), but that is offtopic here and requires some other place of discussion I guess ;)

Revision history for this message
MC Return (mc-return) wrote : Posted in a previous version of this proposal

There is another animations-plug-in that is missing, but currently crashes latest Compiz versions (but did already work in 0.9.x series). It is called simple-animations (Animations that are easy on the eyes). What should we do with that one, Sam ?

Revision history for this message
MC Return (mc-return) wrote : Posted in a previous version of this proposal

I guess animations-add-on was just separated to introduce and define a "Animation Time Step For Intense Effects" which can differ from the "Animation Time Step" usually used, but this function could be moved to one plug-in also, so it won't get lost...

Revision history for this message
Sam Spilsbury (smspillaz) wrote : Posted in a previous version of this proposal

Animationaddon exist so that distributions don't have to ship huge binaries of stuff they don't use ;-)

In any case, I'll get around to looking at the best way to approach this soon. It will take a low priority though over the next few weeks since we really need to ramp up efforts to fix up the OpenGL|ES work before I go on holiday since I won't even have a computer -.- .

Revision history for this message
MC Return (mc-return) wrote : Posted in a previous version of this proposal

Thanks for looking into that issue.
Unfortunately I found out that exactly those plug-ins here, which I merged first, do not seem to work with latest Compiz 0.9.7.8 anymore. :(
I can enable "Animations Experimental" and change settings, but when I select one of those 5 plug-ins (Black Hole, Flicker, Ghost, Popcorn & Raindrop) in "Animations" (for the Open-animation for example) they simply do not work. They do not crash Compiz, but they seem to not be executed either. :(

Revision history for this message
Daniel van Vugt (vanvugt) wrote : Posted in a previous version of this proposal

This branch keeps failing to build (when merged with lp:compiz)

In file included from /home/dan/bzr/compiz/tmp.jc/plugins/animationjc/src/popcorn.cpp:1:0:
/home/dan/bzr/compiz/tmp.jc/plugins/animationjc/src/private.h:6:33: fatal error: composite/composite.h: No such file or directory
compilation terminated.

review: Needs Fixing
Revision history for this message
MC Return (mc-return) wrote : Posted in a previous version of this proposal

I tried to investigate what exactly is the problem here, but did not find out yet.
The strange thing about this error is that composite/composite.h should exist and the other animation plug-ins use almost the same code.
Another strangeness here is the fact that it compiles as standalone plug-in (but does not work then), but does not compile when merged with lp:compiz.

Revision history for this message
MC Return (mc-return) wrote : Posted in a previous version of this proposal

The PLUGINDEPS opengl and composite are missing to fix compilation here.

Unfortunately I am having problems to upload to launchpad since a few days, I hope I'll be able to fix this soonish though.

Revision history for this message
Sam Spilsbury (smspillaz) wrote :

Hey just wanted to say thanks for working on this! It seems like a lot of plugins were lost when git.compiz.org went down, so this is the only mirror I've been able to find of them.

Unmerged revisions

3254. By MC Return

Fixed [-Werror=maybe-uninitialized] compiler warning for the float 'displacement'

3253. By MC Return

Removed plugins/animationjc/VERSION

3252. By MC Return

Added missing PLUGINDEPS composite and opengl to CMakeLists.txt to fix compilation

3251. By MC Return

Rebased on latest lp:compiz (r3285)

3250. By MC Return

Added the unsupported, but fully working plug-in "animationjc" converted from git to bzr (including history) to Compiz 0.9.8.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: