Bug#859461: xterm: seperate package for resize would be nice

[Uwe Kleine-König]

Hello Julien,

On 03/27/2018 11:16 PM, Julien Cristau wrote:
> Control: tag -1 - patch
> 
> On Fri, Mar 23, 2018 at 14:21:01 +0100, Uwe Kleine-König wrote:
> 
>> Control: tag 859461 + patch
>>
>> Hello,
>>
>> On Mon, Apr 03, 2017 at 01:49:32PM -0600, ben hildred wrote:
>>> Which brings me to my request, split resize into a seprate package and 
>>> depend
>>> on it to preserve existing functionality while allowing headless machines to
>>> install just resize.
>> I could make use of this, too. Here is a patch:
>>
> I'm not convinced this split is a good idea.  But:
> 
>> diff --git a/debian/control b/debian/control
>> index 711d5d9b54f2..50d7217da720 100644
>> --- a/debian/control
>> +++ b/debian/control
>> @@ -141,3 +142,13 @@ Description: X terminal emulator
>>   .
>>   Those interested in using koi8rxterm will likely want to install the
>>   xfonts-cyrillic package as well.
>> +
>> +Package: resize
> 
> This isn't a good package name.  It's too generic.

Fair enough. What do you think about termresize? Should the binary be
renamed then, too? (I'd say no.)

>> +Architecture: any
>> +Multi-Arch: foreign
>> +Depends:
>> + ${shlibs:Depends},
>> + ${misc:Depends},
> 
> As-is, this would break upgrades, as it's missing Breaks/Replaces
> against xterm.

Right you are. I can address this after the naming question is resolved.

Best regards
Uwe



signature.asc
Description: OpenPGP digital signature

Bug#859461: xterm: seperate package for resize would be nice

[Thomas Dickey]

On Tue, Mar 27, 2018 at 06:12:03PM -0400, Thomas Dickey wrote:
> On Tue, Mar 27, 2018 at 11:16:55PM +0200, Julien Cristau wrote:
...
> > I'm not convinced this split is a good idea.  But:
> 
> agreed...
> 
> popcon would probably score this as 10.

...unless it's a dependency of xterm, of course.
 
> > > +Architecture: any
> > > +Multi-Arch: foreign
> > > +Depends:
> > > + ${shlibs:Depends},
> > > + ${misc:Depends},
> > 
> > As-is, this would break upgrades, as it's missing Breaks/Replaces
> > against xterm.

...and then you'd have to followup with xterm depending on resize.

-- 
Thomas E. Dickey 
https://invisible-island.net
ftp://ftp.invisible-island.net


signature.asc
Description: Digital signature

Bug#859461: xterm: seperate package for resize would be nice

[Thomas Dickey]

On Tue, Mar 27, 2018 at 11:16:55PM +0200, Julien Cristau wrote:
> Control: tag -1 - patch
> 
> On Fri, Mar 23, 2018 at 14:21:01 +0100, Uwe Kleine-König wrote:
> 
> > Control: tag 859461 + patch
> > 
> > Hello,
> > 
> > On Mon, Apr 03, 2017 at 01:49:32PM -0600, ben hildred wrote:
> > > Which brings me to my request, split resize into a seprate package and 
> > > depend
> > > on it to preserve existing functionality while allowing headless machines 
> > > to
> > > install just resize.
> > I could make use of this, too. Here is a patch:
> > 
> I'm not convinced this split is a good idea.  But:

agreed...

popcon would probably score this as 10.
 
> > diff --git a/debian/control b/debian/control
> > index 711d5d9b54f2..50d7217da720 100644
> > --- a/debian/control
> > +++ b/debian/control
> > @@ -141,3 +142,13 @@ Description: X terminal emulator
> >   .
> >   Those interested in using koi8rxterm will likely want to install the
> >   xfonts-cyrillic package as well.
> > +
> > +Package: resize
> 
> This isn't a good package name.  It's too generic.

:-)
 
> > +Architecture: any
> > +Multi-Arch: foreign
> > +Depends:
> > + ${shlibs:Depends},
> > + ${misc:Depends},
> 
> As-is, this would break upgrades, as it's missing Breaks/Replaces
> against xterm.
> 
> > +Description: Determine size of virtual terminal
> > + This program is used to (re)set the width and height of your current
> > + terminal to the size of the virtual terminal.

The description is another problem...

-- 
Thomas E. Dickey 
https://invisible-island.net
ftp://ftp.invisible-island.net


signature.asc
Description: Digital signature

Bug#859461: xterm: seperate package for resize would be nice

[Julien Cristau]

Control: tag -1 - patch

On Fri, Mar 23, 2018 at 14:21:01 +0100, Uwe Kleine-König wrote:

> Control: tag 859461 + patch
> 
> Hello,
> 
> On Mon, Apr 03, 2017 at 01:49:32PM -0600, ben hildred wrote:
> > Which brings me to my request, split resize into a seprate package and 
> > depend
> > on it to preserve existing functionality while allowing headless machines to
> > install just resize.
> I could make use of this, too. Here is a patch:
> 
I'm not convinced this split is a good idea.  But:

> diff --git a/debian/control b/debian/control
> index 711d5d9b54f2..50d7217da720 100644
> --- a/debian/control
> +++ b/debian/control
> @@ -141,3 +142,13 @@ Description: X terminal emulator
>   .
>   Those interested in using koi8rxterm will likely want to install the
>   xfonts-cyrillic package as well.
> +
> +Package: resize

This isn't a good package name.  It's too generic.

> +Architecture: any
> +Multi-Arch: foreign
> +Depends:
> + ${shlibs:Depends},
> + ${misc:Depends},

As-is, this would break upgrades, as it's missing Breaks/Replaces
against xterm.

> +Description: Determine size of virtual terminal
> + This program is used to (re)set the width and height of your current
> + terminal to the size of the virtual terminal.

Cheers,
Julien

Processed: Re: Bug#859461: xterm: seperate package for resize would be nice

[Debian Bug Tracking System]

Processing control commands:

> tag -1 - patch
Bug #859461 [xterm] xterm: seperate package for resize would be nice
Removed tag(s) patch.

-- 
859461: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=859461
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems

Bug#859461: xterm: seperate package for resize would be nice

[Uwe Kleine-König]

Control: tag 859461 + patch

Hello,

On Mon, Apr 03, 2017 at 01:49:32PM -0600, ben hildred wrote:
> Which brings me to my request, split resize into a seprate package and depend
> on it to preserve existing functionality while allowing headless machines to
> install just resize.
I could make use of this, too. Here is a patch:

diff --git a/debian/changelog b/debian/changelog
index 322f2f59dcfe..f2998d375ef1 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+xterm (331-1ukl1) UNRELEASED; urgency=medium
+
+  * Split resize(1) into a separate package (Closes: #859461)
+
+ -- Uwe Kleine-König   Fri, 23 Mar 2018 14:09:38 +0100
+
 xterm (331-1) unstable; urgency=medium
 
   * New upstream release.
diff --git a/debian/control b/debian/control
index 711d5d9b54f2..50d7217da720 100644
--- a/debian/control
+++ b/debian/control
@@ -115,6 +115,7 @@ Architecture: any
 Multi-Arch: foreign
 Depends:
  xbitmaps,
+ resize,
  ${shlibs:Depends},
  ${misc:Depends},
 Recommends: x11-utils
@@ -141,3 +142,13 @@ Description: X terminal emulator
  .
  Those interested in using koi8rxterm will likely want to install the
  xfonts-cyrillic package as well.
+
+Package: resize
+Architecture: any
+Multi-Arch: foreign
+Depends:
+ ${shlibs:Depends},
+ ${misc:Depends},
+Description: Determine size of virtual terminal
+ This program is used to (re)set the width and height of your current
+ terminal to the size of the virtual terminal.
diff --git a/debian/resize.install b/debian/resize.install
new file mode 100644
index ..04adcfc65a7e
--- /dev/null
+++ b/debian/resize.install
@@ -0,0 +1,2 @@
+usr/bin/resize
+usr/share/man/man1/resize*
diff --git a/debian/xterm.install b/debian/xterm.install
index f4a0bded4143..5961dfa7b0d9 100644
--- a/debian/xterm.install
+++ b/debian/xterm.install
@@ -1,7 +1,7 @@
 etc/X11/app-defaults/*
-usr/bin/*
+usr/bin/*term
 usr/share/doc
-usr/share/man/man1/*
+usr/share/man/man1/*term*
 usr/share/pixmaps
 usr/share/applications/*
 usr/share/icons

Best regards
Uwe


signature.asc
Description: PGP signature

Processed: Re: Bug#859461: xterm: seperate package for resize would be nice

[Debian Bug Tracking System]

Processing control commands:

> tag 859461 + patch
Bug #859461 [xterm] xterm: seperate package for resize would be nice
Added tag(s) patch.

-- 
859461: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=859461
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems

Bug#859461: xterm: seperate package for resize would be nice

[ben hildred]

Package: xterm
Version: 312-2
Severity: wishlist

Dear Maintainer,

For those of us using a machine without a video card a serial console is quite
useful, and in many cases that serial console is controled by xterm or similar
terminal emulator (in my case it is almost always xterm and screen), due to
the continued decrease in availability of hardware serial terminals (not much
of a loss there). now one of the cool new features of using a modern terminal
emulator with rs232 (in adition to utf8, 256 color, sixel graphics (sorry),
etc.) is resizable terminal windows. This is not very dificult as long as you
execute xterm's resize on the remote machine after each resize. (see:
http://unix.stackexchange.com/questions/16578/resizable-serial-console-window)

The one chalenge is in the case of machines that are not running any graphics
software and do not need or want xterm installed, but do want xterm's resize.

Which brings me to my request, split resize into a seprate package and depend
on it to preserve existing functionality while allowing headless machines to
install just resize.