flash operator needs some love

[Diego Iastrubni]

Hola Alejandro,

The problem is that all these patches are tied up together. So, this will be a 
problem... and I am lazy ... :)

Lets split this up into several patches:
patch1:
user, links in www and manager configuration.

patch2 (in the future):
2nd optional configuration.

Both patches will not include modifications to the changelog, just to reduce 
the visual noise.

I will try to have patch1 done today. Do you want me to send it to you 
directly, or to this ML? 
About patch2, IMHO no branching is needed yet, since my code it not working 
yet. Lets see how my work progress in the near future, and decide what to do 
then.

On Wednesday 16 August 2006 21:14, Alejandro Ríos P. wrote:
> El mar, 15-08-2006 a las 10:52 +0300, Diego Iastrubni escribió:
> > Hi,
>
> Hi
>
> > I have been looking into op-panel, and IMHO it needs some work to be done
> > on, as the package is not perfectly functional. I have started working,
> > and I have a semi-working patch (some parts of it you will now like, but
> > I am doing them to ease my work...). The modifications are:
>
> I agree that the package needs some work, thanks for your help. Please
> send small patches and not huge big patches.
>
> > 1) the service runs as root today. this is a big "nono". the patch
> > creates a new user (op-panel) with a home directory (/var/lib/op-panel).
>
> Great.
>
> > 2) the dhtml panel was not working, since variables.txt was not available
> > to it. now the variables.txt is saved in $HOME and linked to the both
> > panels (flash+dhtml)
>
> Great.
>
> > 3) new links are been made in /var/www:
> > 	panel -> /usr/share/op-panel/flash
> > 	dpanel -> /usr/share/op-panel/dhtml
>
> Great.
>
> > 4) i am working on another set of alternative configuration called rapid,
> > which contains auto-detection of extensions. still not working but it
> > will be available "soon"
>
> Please commit this kind of stuff on a separated branch (and not trunk)
> so we can help testing it.
>
> > 5) a manager configuration in /etc/asterisk/manager.d/ which will enable
> > the server to be used out of the box.
>
> I agree.
>
> > 6) many cleanups to the various files, which IMHO make it easier to read
>
> We shall take a look at that ;)
>
> > The patch I am working on can be attached on the next mail if you ask,
> > about 9kb compressed with bz2, or sent directly anyone who asks. It's
> > ugly and contains some hacks (like not building the swf files as the
> > original pkg-voip rules does).  All those hacks will be removed work on
> > the final upload.
>
> Again, please send small and readable patches. I'm clearly interested on
> the changes, since I feel I'm the main responsible for the package.
>
> Thanks and regards.