Re: svn commit: r361066 - head/usr.sbin/jail

2020-05-15 Thread Rodney W. Grimes
> 
> On 5/15/20 4:17 PM, Rodney W. Grimes wrote:
> >> On 5/15/20 3:24 PM, Rodney W. Grimes wrote:
> >>
>  On 5/15/20 6:18 AM, Mateusz Piotrowski wrote:
> 
> > On 5/15/20 1:38 AM, Ryan Moeller wrote:
> >> Author: freqlabs
> >> Date: Thu May 14 23:38:11 2020
> >> New Revision: 361066
> >> URL: https://svnweb.freebsd.org/changeset/base/361066
> >>
> >> Log:
> >>  jail: Add exec.prepare and exec.release command hooks
> >>  
> >>  This change introduces new jail command hooks that run before and 
> >> after any
> >>  other actions.
> > Should it go into RELNOTES?
>  I'm not sure what all the criteria are for relnotes.
>  The committer's guide makes it seem like relnotes is for breaking
>  changes, which this is not.
> >>> Please could you point at which specific language in the commiters
> >>> guide makes you believe that the RELNOTES are for breaking changes?
> >> Every mention of "release notes" in the document is in the context of
> >> deprecating, removing,
> >> or breaking things, with one exception:
> > Fair, there should be a section on "new features and enhnacements"
> > which is laking.  However if one reads a release notes from a shipping
> > version it becomes clear that the actual majority of the text in it is
> > "new stuff."
> 
> Now that I know better, how do I retcon this and other potentially 
> relnoteworthy enhancements I've made? :)

There is a top level file in the src tree called RELNOTES that
you can use to ad an entry to describing the change and the
revision number.

> >>   > Relnotes:??? If the change is a candidate for inclusion in the
> >> release notes for the next release from the branch, set to yes.
> >>> RELNOTES should be for all changes that have user visible impact
> >>> of any type.
> >>>
>  -Ryan
> 

-- 
Rod Grimes rgri...@freebsd.org
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r361066 - head/usr.sbin/jail

2020-05-15 Thread Ryan Moeller



On 5/15/20 4:17 PM, Rodney W. Grimes wrote:

On 5/15/20 3:24 PM, Rodney W. Grimes wrote:


On 5/15/20 6:18 AM, Mateusz Piotrowski wrote:


On 5/15/20 1:38 AM, Ryan Moeller wrote:

Author: freqlabs
Date: Thu May 14 23:38:11 2020
New Revision: 361066
URL: https://svnweb.freebsd.org/changeset/base/361066

Log:
 jail: Add exec.prepare and exec.release command hooks
 
 This change introduces new jail command hooks that run before and after any

 other actions.

Should it go into RELNOTES?

I'm not sure what all the criteria are for relnotes.
The committer's guide makes it seem like relnotes is for breaking
changes, which this is not.

Please could you point at which specific language in the commiters
guide makes you believe that the RELNOTES are for breaking changes?

Every mention of "release notes" in the document is in the context of
deprecating, removing,
or breaking things, with one exception:

Fair, there should be a section on "new features and enhnacements"
which is laking.  However if one reads a release notes from a shipping
version it becomes clear that the actual majority of the text in it is
"new stuff."


Now that I know better, how do I retcon this and other potentially 
relnoteworthy enhancements I've made? :)



  > Relnotes:??? If the change is a candidate for inclusion in the
release notes for the next release from the branch, set to yes.

RELNOTES should be for all changes that have user visible impact
of any type.


-Ryan

___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r361066 - head/usr.sbin/jail

2020-05-15 Thread Rodney W. Grimes
> On 5/15/20 3:24 PM, Rodney W. Grimes wrote:
> 
> >> On 5/15/20 6:18 AM, Mateusz Piotrowski wrote:
> >>
> >>> On 5/15/20 1:38 AM, Ryan Moeller wrote:
>  Author: freqlabs
>  Date: Thu May 14 23:38:11 2020
>  New Revision: 361066
>  URL: https://svnweb.freebsd.org/changeset/base/361066
> 
>  Log:
>  jail: Add exec.prepare and exec.release command hooks
>  
>  This change introduces new jail command hooks that run before and 
>  after any
>  other actions.
> >>> Should it go into RELNOTES?
> >> I'm not sure what all the criteria are for relnotes.
> >> The committer's guide makes it seem like relnotes is for breaking
> >> changes, which this is not.
> > Please could you point at which specific language in the commiters
> > guide makes you believe that the RELNOTES are for breaking changes?
> 
> Every mention of "release notes" in the document is in the context of 
> deprecating, removing,
> or breaking things, with one exception:

Fair, there should be a section on "new features and enhnacements"
which is laking.  However if one reads a release notes from a shipping
version it becomes clear that the actual majority of the text in it is
"new stuff."

>  > Relnotes:??? If the change is a candidate for inclusion in the 
> release notes for the next release from the branch, set to yes.
> >
> > RELNOTES should be for all changes that have user visible impact
> > of any type.
> >
> >> -Ryan
> 

-- 
Rod Grimes rgri...@freebsd.org
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r361066 - head/usr.sbin/jail

2020-05-15 Thread Ryan Moeller

On 5/15/20 3:24 PM, Rodney W. Grimes wrote:


On 5/15/20 6:18 AM, Mateusz Piotrowski wrote:


On 5/15/20 1:38 AM, Ryan Moeller wrote:

Author: freqlabs
Date: Thu May 14 23:38:11 2020
New Revision: 361066
URL: https://svnweb.freebsd.org/changeset/base/361066

Log:
jail: Add exec.prepare and exec.release command hooks

This change introduces new jail command hooks that run before and after any

other actions.

Should it go into RELNOTES?

I'm not sure what all the criteria are for relnotes.
The committer's guide makes it seem like relnotes is for breaking
changes, which this is not.

Please could you point at which specific language in the commiters
guide makes you believe that the RELNOTES are for breaking changes?


Every mention of "release notes" in the document is in the context of 
deprecating, removing,

or breaking things, with one exception:

> Relnotes:    If the change is a candidate for inclusion in the 
release notes for the next release from the branch, set to yes.


RELNOTES should be for all changes that have user visible impact
of any type.


-Ryan

___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r361066 - head/usr.sbin/jail

2020-05-15 Thread Rodney W. Grimes
> On 5/15/20 6:18 AM, Mateusz Piotrowski wrote:
> 
> > On 5/15/20 1:38 AM, Ryan Moeller wrote:
> >> Author: freqlabs
> >> Date: Thu May 14 23:38:11 2020
> >> New Revision: 361066
> >> URL: https://svnweb.freebsd.org/changeset/base/361066
> >>
> >> Log:
> >>jail: Add exec.prepare and exec.release command hooks
> >>
> >>This change introduces new jail command hooks that run before and after 
> >> any
> >>other actions.
> > Should it go into RELNOTES?
> I'm not sure what all the criteria are for relnotes.
> The committer's guide makes it seem like relnotes is for breaking 
> changes, which this is not.

Please could you point at which specific language in the commiters
guide makes you believe that the RELNOTES are for breaking changes?

RELNOTES should be for all changes that have user visible impact
of any type.

> -Ryan

-- 
Rod Grimes rgri...@freebsd.org
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r361066 - head/usr.sbin/jail

2020-05-15 Thread Ryan Moeller

On 5/15/20 6:18 AM, Mateusz Piotrowski wrote:


On 5/15/20 1:38 AM, Ryan Moeller wrote:

Author: freqlabs
Date: Thu May 14 23:38:11 2020
New Revision: 361066
URL: https://svnweb.freebsd.org/changeset/base/361066

Log:
   jail: Add exec.prepare and exec.release command hooks
   
   This change introduces new jail command hooks that run before and after any

   other actions.

Should it go into RELNOTES?

I'm not sure what all the criteria are for relnotes.
The committer's guide makes it seem like relnotes is for breaking 
changes, which this is not.


-Ryan

___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r361066 - head/usr.sbin/jail

2020-05-15 Thread Mateusz Piotrowski
On 5/15/20 1:38 AM, Ryan Moeller wrote:
> Author: freqlabs
> Date: Thu May 14 23:38:11 2020
> New Revision: 361066
> URL: https://svnweb.freebsd.org/changeset/base/361066
>
> Log:
>   jail: Add exec.prepare and exec.release command hooks
>   
>   This change introduces new jail command hooks that run before and after any
>   other actions.
Should it go into RELNOTES?
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"