Bundled Flash

[Adam Williamson]

On Fri, 2013-08-23 at 06:36 -0700, T.C. Hollingsworth wrote:
> On Fri, Aug 23, 2013 at 5:40 AM, Adam Williamson <awilliam at redhat.com> wrote:
> > All the upstream projects I found seemed to consider jumping to tinymce
> > 4 a rather large move. Debian packages 3 and 4 as separate packages. I
> > rather think we should do the same rather than just pretend they're the
> > same thing and we'll be easily able to migrate things from one to the
> > other.
> 
> I should have known it couldn't be that easy.  :-/
> 
> > Or, at least let's get a tinymce 3.5.8 built and all things that bundle
> > 3 un-bundled before we start worrying about 4. :)
> >
> > So, after some teeth gnashing, I've confirmed roundcube can indeed be
> > relatively easily unbundled to use a system 3.5.8 package. It needs the
> > spellchecker plugin which is a separate package in Fedora, which tripped
> > me up for a while.
> >
> > However, to do this, I run into the fucking
> > convert-a-directory-to-a-symlink old chestnut, and RPM/yum just isn't
> > having it. Following the breadcrumbs all over this list and Bugzilla I
> > came up with this %pretrans:
> >
> > %pretrans -p <lua>
> > -- changed dir to symlink in 0.9.3-1, workaround RPM issues
> > path = "%{roundcubedir}/program/js/tiny_mce"
> > if posix.readlink(path) then
> > os.remove(path)
> > end
> >
> > But yum/rpm just won't wear it. 'yum update mynewroundcubepackage.rpm'
> > fails with a file conflict just like it does if there's no %pretrans at
> > all. I was able to test my unbundling simply by manually removing and
> > reinstalling the roundcube package, but I can't push it out anywhere
> > unless I can get a %pretrans which yum/rpm will be happy with. Anyone?
> > https://bugzilla.redhat.com/show_bug.cgi?id=975909 may be involved, I
> > guess.
> 
> I believe that's the code for the symlink->directory direction.  The state of
> the art for directory->symlink seems to be:
> https://bugs.launchpad.net/rpm/+bug/633636/comments/3
> as linked to in bug:
> https://bugzilla.redhat.com/show_bug.cgi?id=447156
> 
> And it's god awful.  How much beer do I have to buy Panu to make this work
> properly?

Oh, god, yes, I saw that, and assumed it was an earlier cut.

> You could just say "screw anaconda installs" and just use `rm -rf`.  Though I
> tried that once, in *EPEL* even, and it only took a week for someone to
> complain. :-(

I suppose we could do it that way wrapped in an 'if' statement so it
only runs on updates; seems reasonable to assume bash is available for
an update transaction?

> > roundcubemail ships tinymce
> > as /usr/share/roundcubemail/program/js/tiny_mce and hooks into it rather
> > untidily; it seems rather cleaner to just do 'ln
> > -s /usr/share/tinymce/jscripts/tiny_mce
> > %{buildroot}%{roundcubedir}/program/js/tiny_mce' in the spec rather than
> > write a messy patch for roundcube to change where it looks for tinymce,
> > which would probably need constant updating.
> 
> Any chance you could just use an Alias in the apache config?  Then you can just
> delete the directory and not muck around with making yum happy.

Doesn't seem to work. Seems like it's just ignored: if I set it and move
tiny_mce/ away, no tinymce. If I leave it set but also put the tiny_mce
directory in place on the filesystem (by symlink or not), tinymce works.
I don't actually have any idea whether PHP is supposed to respect the
Apache aliases or whether it's looking at the 'real' filesystem, but it
seems to be b).
-- 
Adam Williamson
Fedora QA Community Monkey
IRC: adamw | Twitter: AdamW_Fedora | identi.ca: adamwfedora
http://www.happyassassin.net