<div dir="ltr">Hi Yaroslav<div><br></div><div>I do not mind pushing the changes directly, but I thought you may want to check things before as this was the first thing I did in this project. Will push now.</div><div><br></div><div>I think both server packages can make use of the -common package.</div><div><br></div><div>Here I have a thought, why do we have separate server packages? I know the FTP masters do not want unnecessary split of the packages. Is there any specific reason why we have different pages for the scraping server, the normal server and the library?</div><div>Just checking what your thoughts were regarding this.</div><div><br></div><div>Some facts:</div><div>- common ~60 KB</div><div>- scraping ~180 KB</div><div>- standalone ~1.1 MB</div><div>- xorg extension ~200 KB</div><div><br></div><div>The dependencies are quite similar too. The main difference is that the extension do not need perl dependencies.</div><div><br></div><div>My suggestion is that we merge them together, but I do not want to do that without your (common) approval :-)</div><div><br></div><div>Best regards</div><div><br></div><div>// Ola</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 26, 2016 at 1:12 AM, Yaroslav Halchenko <span dir="ltr"><<a href="mailto:debian@onerussian.com" target="_blank">debian@onerussian.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
On Mon, 25 Jan 2016, Ola Lundqvist wrote:<br>
<br>
</span><span class="">> I have started to review the control file. I have attached a change<br>
> proposal based on my experience with vnc packaging from before.<br>
<br>
</span>we could try continuing reviewing via email, or should we may be push it<br>
also on github and then use PR/comments mechanism?<br>
<br>
otherwise -- we just usually trusted each other's judgement so far ;)<br>
<br>
some comments below<br>
<span class=""><br>
> @@ -112,14 +113,14 @@ Depends:<br>
> >  Conflicts:<br>
> >   tigervnc-server (<< 1.1.90),<br>
> >   tigervnc-viewer (<< 1.1.90)<br>
> > -Description: Virtual network computing; Common software needed by clients<br>
> > and servers<br>
> > +Description: Virtual network computing; Common software needed by servers<br>
> >   VNC stands for Virtual Network Computing. It is, in essence, a remote<br>
> >   display system which allows you to view a computing `desktop' environment<br>
> >   not only on the machine where it is running, but from anywhere on the<br>
> >   Internet and from a wide variety of machine architectures.<br>
> >   .<br>
> >   It is implemented in a client/server model. This package provides the<br>
> > common<br>
> > - software for both client and server.<br>
> > + software for servers.<br>
<br>
> >  Package: tigervnc-scraping-server<br>
> >  Architecture: any<br>
<br>
<br>
> The common tools are not needed by the viewer. The vncconfig tool is just<br>
> to manage a Xvnc (or X with extension) server and the vncpasswd command is<br>
> also just needed on the server side. Or am I missing something here?<br>
<br>
</span>that may be they could be used by both server packages?<br>
<br>
so we may be should dump it from viewer package depends indeed<br>
<span class=""><br>
> I guess this packaging practice is a leftover from where tightvnc and vnc4<br>
> actually had a common package. Now the need arise again but just because we<br>
> provide serveral server alternatives.<br>
<br>
> @@ -131,7 +132,6 @@ Depends:<br>
> >   ${shlibs:Depends},<br>
> >   ${misc:Depends},<br>
> >  Recommends: libgl1-mesa-dri (>= 7.1~rc1)<br>
> > -Suggests: xfonts-100dpi | xfonts-75dpi, xfonts-scalable<br>
> >  Provides:<br>
> >   vnc-server,<br>
> >   ${videoabi},<br>
<br>
> I do not think the scraping server need fonts on its own, or?<br>
<br>
</span>to say the truth -- don't know for sure.  logically -- indeed it<br>
shouldn't need them<br>
<span class=""><br>
> The below patch section is actually commented within the section itself as<br>
> there are so many things that I propose.<br>
<br>
> @@ -160,13 +160,15 @@ Depends:<br>
> >   keyboard-configuration [linux-any kfreebsd-any],<br>
> >   udev (>= 149) [linux-any],<br>
> >   tigervnc-common,<br>
> > + xauth,<br>
> >   ${shlibs:Depends},<br>
> >   ${misc:Depends},<br>
> >   ${perl:Depends}<br>
<br>
<br>
> The vncserver command actually depend on xauth so it should be here. I<br>
> could accept a recommends as it is just the vncserver command that need it.<br>
> The Xvnc command do not. However a dependency here is not really a problem<br>
> as one would expect it to be there.<br>
<br>
</span>ok<br>
<span class=""><br>
> -Recommends: libgl1-mesa-dri (>= 7.1~rc1)<br>
> > +Recommends: libgl1-mesa-dri (>= 7.1~rc1), xfonts-base, x11-xserver-utils<br>
> >  Suggests: xfonts-100dpi | xfonts-75dpi, xfonts-scalable<br>
<br>
<br>
> xfonts-base is just to recommend a standard set of fonts. Previous vnc<br>
> packages have had it but maybe it is not needed.<br>
<br>
</span>IDK (I don't know or I don't care) ;-)<br>
<span class=""><br>
> The recommendation on x11-server-utils is to get tools like xrandr, xauth<br>
> and the like. They are quite useful... But maybe the contents have changed<br>
> too many times since first added to the vnc suite.<br>
<br>
</span>ok<br>
<span class=""><br>
>  Provides:<br>
> >   vnc-server,<br>
> > + xserver,<br>
> >   ${videoabi},<br>
> >   ${inputabi}<br>
> >  Replaces: tigervnc-server (<< 1.1.90)<br>
<br>
<br>
> I think the vncserver package should declare that it is its own xserver.<br>
> Not sure if this break some rule though. Tightvnc and vnc4 so so and I have<br>
> not got any complaints about this.<br>
<br>
</span>ok<br>
<span class=""><br>
> @@ -193,7 +195,7 @@ Depends:<br>
> >   tigervnc-common,<br>
> >   ${shlibs:Depends},<br>
> >   ${misc:Depends},<br>
> > -Recommends: libgl1-mesa-dri (>= 7.1~rc1)<br>
> > +Recommends: libgl1-mesa-dri (>= 7.1~rc1), xfonts-base, x11-xserver-utils<br>
> >  Suggests: xfonts-100dpi | xfonts-75dpi, xfonts-scalable<br>
> >  Provides:<br>
> >   vnc-server,<br>
<br>
<br>
> Same comment as for the standalone server. Extension have same kind of<br>
> need. However it is not its own xserver so it should not provide it.<br>
<br>
> @@ -215,11 +217,10 @@ Package: tigervnc-viewer<br>
> >  Architecture: any<br>
> >  Depends:<br>
> >   xserver-common (>= 2:1.7.7),<br>
> > - tigervnc-common,<br>
> >   ${shlibs:Depends},<br>
> >   ${misc:Depends},<br>
<br>
<br>
> The viewer do not need vncpasswd nor vncconfig. So I think we can remove<br>
> it. If you know a use-case when it should be needed then please let me know.<br>
<br>
</span>ok<br>
<span class=""><br>
>  Provides:<br>
> > - vnc-viewer<br>
> > + vncviewer, vnc-viewer<br>
> >  Description: Virtual network computing client software for X<br>
> >   VNC stands for Virtual Network Computing. It is, in essence, a remote<br>
> >   display system which allows you to view a computing `desktop' environment<br>
<br>
<br>
> Historically the vnc4 and tightvnc packages have provided this vncviewer<br>
> one too. However that is because we had a realvnc/vnc/vnc3 package many<br>
> years ago that had this alias. Maybe we do not need to support transition<br>
> cases that dates back 8 years or so... :-) I added it anyway.<br>
<br>
> What do you think?<br>
<br>
</span>ok<br>
;-)<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
Yaroslav O. Halchenko<br>
Center for Open Neuroscience     <a href="http://centerforopenneuroscience.org" rel="noreferrer" target="_blank">http://centerforopenneuroscience.org</a><br>
Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755<br>
Phone: <a href="tel:%2B1%20%28603%29%20646-9834" value="+16036469834">+1 (603) 646-9834</a>                       Fax: <a href="tel:%2B1%20%28603%29%20646-1419" value="+16036461419">+1 (603) 646-1419</a><br>
WWW:   <a href="http://www.linkedin.com/in/yarik" rel="noreferrer" target="_blank">http://www.linkedin.com/in/yarik</a><br>
</div></div></blockquote></div><br></div>