[mythtv] Re: compiling mythtv frontend for cygwin
Mario L
superm1 at gmail.com
Thu Oct 14 14:38:11 UTC 2004
> On Tuesday 12 October 2004 10:51 am, Geoffrey Hausheer wrote:
> > On Tue, 12 Oct 2004 03:47:43 -0500, Mario L wrote:
> > > .... If that patch could get applied to CVS by someone, then
> > > some other people can finally join in on this since it is basically
> > > down to a science.
> >
> > While Isaac is probably the only one with a say in whether something
> > like this goes in, I wouldn't count on it. First off, there is no
> > reason to put it into the code yet, as it doesn't work. Anyone
> > competent enough to do hacking on it can install your patch and make
> > new ones. Second the code will be nearly impossible to maintain.
> > There are ifdefs all over the place. The ones controlling headers I
> > can understand, but there are many others. Since almost all of the
> > developers run Linux, they don't want to worry about whether they are
> > breaking the win32 build, and I would guess that Isaac doesn't want to
> > have to wait for the win32 maintainers to fix bugs before releasing
> > new versions. I would say (a) get it working first. I don't see nay
> > compelling reason to add it otherwise, and (b) after you do that,
> > clean up the patch to minimize differences such that the maintenance
> > will be easier.
> >
> > As an aside, if you want unix people to look at your patches .tar.bz2
> > (or .gz) is a much better format that rar. I hate having to use
> > proprietary programs to just look at a patch. And again anyone
> > familiar with cygwin will know how to use tar, so you aren't
> > alienating your users.
>I agree with all this - there is too much cruft in the patch that should be
>slimmed down by quite a bit before I'd consider applying it.
Okay sounds like a plan, once I actually get it to a point of working,
I will clean it up. Also I'll remember to release it in a more unix
friendly format next time. Thanks for the input.
More information about the mythtv-dev
mailing list