Bug#947233: ifupdown: Please include restart-ms option in can config

2021-10-12 Thread Santiago Ruano Rincón
Control: tags -1 + pending

El 11/10/21 a las 17:44, Joerg Dorchain escribió:
> On Mon, Oct 11, 2021 at 05:14:15PM +0200, Santiago Ruano Rincón wrote:
> > 
> > Hi,
> > 
> > On Mon, 23 Dec 2019 11:03:33 +0100 Joerg Dorchain  
> > wrote:
> > > Package: ifupdown
> > > Version: 0.8.35
> > > Tags: Patch
> > > Severity: wishlist
> > > 
> > > Hi,
> > > 
> > > please include the restart-ms option in the can static method, e.g as per
> > > attached patch.
> > > 
> > > Thanks for considering.
> > > 
> > > Bye,
> > > 
> > > Joerg
> > > 
> > > 
> > > --- ifupdown-0.8.35.orig/can.defn 2018-10-29 22:29:33.0 +0100
> > > +++ ifupdown-0.8.35/can.defn  2019-12-23 11:00:03.421389325 +0100
> > > @@ -14,6 +14,7 @@
> > >  triple triple-- activate triple sampling (on|off)
> > >  oneshot oneshot  -- one shot mode (on|off)
> > >  berr berr-- activate berr reporting (on|off) 
> > > +restart-ms restart-ms-- restart-ms (0..)
> > >  
> > >up
> > >  ip link set %iface% type can bitrate %bitrate%
> > > @@ -22,6 +23,7 @@
> > >  [[ ip link set %iface% type can triple-sampling %triple% ]]
> > >  [[ ip link set %iface% type can one-shot %oneshot% ]]
> > >  [[ ip link set %iface% type can berr-reporting %berr% ]]
> > > +[[ ip link set %iface% type can restart-ms %restart-ms% ]]
> > >  ip link set %iface% up
> > >  
> > >down
> > > 
> > 
> > Thanks for your patch. I cannot test it by myself but I suppose it is OK
> > to include it. However, could you please document this option in your
> > patch? You may include it in interfaces(5)
> 
> Here you are:
> --- interfaces.5.orig   2021-10-11 17:39:09.417498419 +0200
> +++ interfaces.52021-10-11 17:42:08.263610653 +0200
> @@ -889,6 +889,9 @@
>  .TP
>  .BI berr " berr"
>  activate berr reporting (on|off)
> +.TP
> +.BI restart-ms " restart-ms"
> +set restart-ms value (0..)
>  .RE
>  .SH KNOWN BUGS/LIMITATIONS
>  The

Thanks! And sorry, the man patch was not really needed. I belatedly
realised realised that section was generated from the definition.

Cheers,

 -- Santiago



signature.asc
Description: PGP signature


Bug#947233: ifupdown: Please include restart-ms option in can config

2021-10-11 Thread Joerg Dorchain
On Mon, Oct 11, 2021 at 05:14:15PM +0200, Santiago Ruano Rincón wrote:
> 
> Hi,
> 
> On Mon, 23 Dec 2019 11:03:33 +0100 Joerg Dorchain  wrote:
> > Package: ifupdown
> > Version: 0.8.35
> > Tags: Patch
> > Severity: wishlist
> > 
> > Hi,
> > 
> > please include the restart-ms option in the can static method, e.g as per
> > attached patch.
> > 
> > Thanks for considering.
> > 
> > Bye,
> > 
> > Joerg
> > 
> > 
> > --- ifupdown-0.8.35.orig/can.defn   2018-10-29 22:29:33.0 +0100
> > +++ ifupdown-0.8.35/can.defn2019-12-23 11:00:03.421389325 +0100
> > @@ -14,6 +14,7 @@
> >  triple triple  -- activate triple sampling (on|off)
> >  oneshot oneshot-- one shot mode (on|off)
> >  berr berr  -- activate berr reporting (on|off) 
> > +restart-ms restart-ms  -- restart-ms (0..)
> >  
> >up
> >  ip link set %iface% type can bitrate %bitrate%
> > @@ -22,6 +23,7 @@
> >  [[ ip link set %iface% type can triple-sampling %triple% ]]
> >  [[ ip link set %iface% type can one-shot %oneshot% ]]
> >  [[ ip link set %iface% type can berr-reporting %berr% ]]
> > +[[ ip link set %iface% type can restart-ms %restart-ms% ]]
> >  ip link set %iface% up
> >  
> >down
> > 
> 
> Thanks for your patch. I cannot test it by myself but I suppose it is OK
> to include it. However, could you please document this option in your
> patch? You may include it in interfaces(5)

Here you are:
--- interfaces.5.orig   2021-10-11 17:39:09.417498419 +0200
+++ interfaces.52021-10-11 17:42:08.263610653 +0200
@@ -889,6 +889,9 @@
 .TP
 .BI berr " berr"
 activate berr reporting (on|off)
+.TP
+.BI restart-ms " restart-ms"
+set restart-ms value (0..)
 .RE
 .SH KNOWN BUGS/LIMITATIONS
 The

Bye,

Joerg


signature.asc
Description: PGP signature


Bug#947233: ifupdown: Please include restart-ms option in can config

2021-10-11 Thread Santiago Ruano Rincón
Hi,

On Mon, 23 Dec 2019 11:03:33 +0100 Joerg Dorchain  wrote:
> Package: ifupdown
> Version: 0.8.35
> Tags: Patch
> Severity: wishlist
> 
> Hi,
> 
> please include the restart-ms option in the can static method, e.g as per
> attached patch.
> 
> Thanks for considering.
> 
> Bye,
> 
> Joerg
> 
> 
> --- ifupdown-0.8.35.orig/can.defn 2018-10-29 22:29:33.0 +0100
> +++ ifupdown-0.8.35/can.defn  2019-12-23 11:00:03.421389325 +0100
> @@ -14,6 +14,7 @@
>  triple triple-- activate triple sampling (on|off)
>  oneshot oneshot  -- one shot mode (on|off)
>  berr berr-- activate berr reporting (on|off) 
> +restart-ms restart-ms-- restart-ms (0..)
>  
>up
>  ip link set %iface% type can bitrate %bitrate%
> @@ -22,6 +23,7 @@
>  [[ ip link set %iface% type can triple-sampling %triple% ]]
>  [[ ip link set %iface% type can one-shot %oneshot% ]]
>  [[ ip link set %iface% type can berr-reporting %berr% ]]
> +[[ ip link set %iface% type can restart-ms %restart-ms% ]]
>  ip link set %iface% up
>  
>down
> 

Thanks for your patch. I cannot test it by myself but I suppose it is OK
to include it. However, could you please document this option in your
patch? You may include it in interfaces(5)

Thanks,

 -- Santiago



Bug#947233: ifupdown: Please include restart-ms option in can config

2019-12-23 Thread Joerg Dorchain
Package: ifupdown
Version: 0.8.35
Tags: Patch
Severity: wishlist

Hi,

please include the restart-ms option in the can static method, e.g as per
attached patch.

Thanks for considering.

Bye,

Joerg


--- ifupdown-0.8.35.orig/can.defn   2018-10-29 22:29:33.0 +0100
+++ ifupdown-0.8.35/can.defn2019-12-23 11:00:03.421389325 +0100
@@ -14,6 +14,7 @@
 triple triple  -- activate triple sampling (on|off)
 oneshot oneshot-- one shot mode (on|off)
 berr berr  -- activate berr reporting (on|off) 
+restart-ms restart-ms  -- restart-ms (0..)
 
   up
 ip link set %iface% type can bitrate %bitrate%
@@ -22,6 +23,7 @@
 [[ ip link set %iface% type can triple-sampling %triple% ]]
 [[ ip link set %iface% type can one-shot %oneshot% ]]
 [[ ip link set %iface% type can berr-reporting %berr% ]]
+[[ ip link set %iface% type can restart-ms %restart-ms% ]]
 ip link set %iface% up
 
   down