[Xprint] Re: Newer version of the combined video+print serverfrom?

[Roland Mainz]

Julien Lafon wrote:
> 
> On Apr 3, 2005 6:01 AM, Roland Mainz <roland.mainz at nrubsig.org> wrote:
> > Roland Mainz wrote:
> > > > > > > > Roland - had you time to continue your work on the combined
> > > > > > > > ('unified') video+print server patch yet?
> > > > > > >
> > > > > > > Unfortunately no (except the integration of the fixes you send me) ...
> > > > > > > almost every minute of my spare free time was consumed by my duties as
> > > > > > > release manager of the X11R6.8.2 release (and then I was at the
> > > > > > > XDevconf+LinuxWorld in Boston) ...
> > > > > >
> > > > > > Attached is a new version of the patch for testing. It deals with most
> > > > > > of the problems of the previous version except that there is no switch
> > > > > > to define a server mode yet (e.g. "video", "print", "unified").
> > > > >  ===================================================================
> > > > > > RCS file: /cvs/xorg/xc/config/cf/Imake.rules,v
> > > > > > retrieving revision 1.8
> > > > > > diff -u -2 -0 -r1.8 Imake.rules
> > > > > > --- xc/config/cf/Imake.rules    1 Feb 2005 22:27:00 -0000       1.8
> > > > > > +++ xc/config/cf/Imake.rules    5 Mar 2005 20:12:50 -0000
> > > > > > @@ -3529,40 +3529,66 @@
> > > > > The patch does not apply to XOrg head. Am I doing something wrong here?
> > > >
> > > > Uhm... uhm... that's likely my fault as I have dismanteled the patch and
> > > > already commited smaller chunks of it to Xorg trunk (like
> > > > https://bugs.freedesktop.org/show_bug.cgi?id=2653) ... either wait until
> > > > I have a new patch or pull-by date (e.g. % cvs -z9 checkout -D "Fri Mar
> > > > 4 23:38:24 MET 2005" xc # should pull a version of the tree where the
> > > > patch applies correctly...) ...
> > >
> > > Attached is the next cycle of the patch. As usual you have to compile it
> > > with
> > > - #define PrintOnlyServer NO
> > > to enable the "unified" (video+print) display
> > >   and
> > > - #define DoLoadableServer NO
> > > as the Xprint DDXs are no loadable modules yet and have some dependicies
> > > (mainly the Postscript DDX which hooks deeply into the Freetype2 font
> > > module)
> >
> > For the log:
> > I've commited an updated version of the patch to Xorg trunk (see
> > attached commit log). However there is a dispute about the dependicies
> > the patch has introduced which require some adjustments or even a
> > backout... ;-(
> Roland: What is the exact matter of the dispute?

Check https://bugs.freedesktop.org/show_bug.cgi?id=2792 (or your
BugZilla email), the primary complain seem to be that the DIX code
(xc/progras/Xserver/dix/) now depends on DiPrint.h (which -
theoretically - shouldn't be done that way since some people want to
compile the tree with |BuildXprint| set to |NO|) and that I have added
stub (the |Printer*()|) functions for all servers which do not have
print functionality. I discussed the matter with Egbert Eich/SuSE and
posted two versions of a fix, hopefully one of them will be accepted...

----

Bye,
Roland

-- 
  __ .  . __
 (o.\ \/ /.o) roland.mainz at nrubsig.org
  \__\/\/__/  MPEG specialist, C&&JAVA&&Sun&&Unix programmer
  /O /==\ O\  TEL +49 641 7950090
 (;O/ \/ \O;)