[mythtv] Re: [mythtv-commits] mythtv commit: r7932 by nigel

[Nigel Pearson]

> TVRec::GetScreenGrab() is a static function and could be moved to
> the preview generator.

	Until we get a "proper" solution,
I have just put this in previewgenerator.cpp :
#ifdef USING_BACKEND
...
#else
     VERBOSE(VB_IMPORTANT,
             "Backend code not compiled in - cannot generate preview.");
#endif



> But as an aside I never intended the --disable-backend and
> --disable-frontend params to be used in perpetuity.

	In this case, we aren't actually using the --disable.
It is just not explicitly enabled on Darwin, and thus we
are using the default of backend="no" on line 76

	When I originally added the "backend" variable way back
(patch.macosx.23, which Isaac applied, now r4141 in SVN),
I was also intending it to be for faster building too.
So, if you only want to build stuff to install onto a
frontend machine, you comment backend out of setting.pro


	There isn't a lot of difference in the build time
between libmythtv for frontend versus frontend+backend
(expecially compared to the time for libav* and *mpeg2),
so it wouldn't be a big deal to lose this.

	I would like to keep the clean "this is frontend-only,
this is backend-only, and this is common" division, though.

--
Nigel Pearson, nigel at ind.tansu.com.au|"Its not easy having a good time
Telstra Dev. Lab, Sydney, Australia  | ...
Office: 9814 4803    Fax:  9814 4897 | even smiling makes my face ache!"
Mobile: 0408 664435  Home: 9792 6998 |     Dr Frank N Furter