public inbox for cygwin-patches@cygwin.com
 help / color / mirror / Atom feed
* Where should relnote updates for Cygwin DLL patches be going?
@ 2023-07-11  8:05 Mark Geisert
  2023-07-11  8:21 ` Corinna Vinschen
  0 siblings, 1 reply; 4+ messages in thread
From: Mark Geisert @ 2023-07-11  8:05 UTC (permalink / raw)
  To: cygwin-patches

AIUI for cygwin-3_4-branch they currently go to release/3.4.8.
For the main|master branch they currently go where?
I hope to get it right the first time ;-).
Thank you,

..mark

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Where should relnote updates for Cygwin DLL patches be going?
  2023-07-11  8:05 Where should relnote updates for Cygwin DLL patches be going? Mark Geisert
@ 2023-07-11  8:21 ` Corinna Vinschen
  2023-07-11  8:49   ` Mark Geisert
  0 siblings, 1 reply; 4+ messages in thread
From: Corinna Vinschen @ 2023-07-11  8:21 UTC (permalink / raw)
  To: cygwin-patches

On Jul 11 01:05, Mark Geisert wrote:
> AIUI for cygwin-3_4-branch they currently go to release/3.4.8.
> For the main|master branch they currently go where?

release/3.5.0

An entry there is only necessary if it doesn't get picked for 3.4
anyway.

> I hope to get it right the first time ;-).

Is the release model confusing?  If so, can you explain why?


Thanks,
Corinna

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Where should relnote updates for Cygwin DLL patches be going?
  2023-07-11  8:21 ` Corinna Vinschen
@ 2023-07-11  8:49   ` Mark Geisert
  2023-07-11  9:45     ` Corinna Vinschen
  0 siblings, 1 reply; 4+ messages in thread
From: Mark Geisert @ 2023-07-11  8:49 UTC (permalink / raw)
  To: cygwin-patches

Hi Corinna,

Corinna Vinschen wrote:
> On Jul 11 01:05, Mark Geisert wrote:
>> AIUI for cygwin-3_4-branch they currently go to release/3.4.8.
>> For the main|master branch they currently go where?
> 
> release/3.5.0
> 
> An entry there is only necessary if it doesn't get picked for 3.4
> anyway.

Ah, that helps me understand.

>> I hope to get it right the first time ;-).
> 
> Is the release model confusing?  If so, can you explain why?

I think I haven't been paying close enough attention and have been doing the 
relnote updates by rote.  But there being two active branches and I 
(understandably) don't determine which releases my commits go to means I should 
wait until they show up on the cvs-patches list, then I will know which relnote 
files to update.  That should work OK, right?

Is it preferred that relnote updates should be separate patches from the code updates?
Thanks,

..mark

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Where should relnote updates for Cygwin DLL patches be going?
  2023-07-11  8:49   ` Mark Geisert
@ 2023-07-11  9:45     ` Corinna Vinschen
  0 siblings, 0 replies; 4+ messages in thread
From: Corinna Vinschen @ 2023-07-11  9:45 UTC (permalink / raw)
  To: cygwin-patches

On Jul 11 01:49, Mark Geisert wrote:
> Hi Corinna,
> 
> Corinna Vinschen wrote:
> > On Jul 11 01:05, Mark Geisert wrote:
> > > AIUI for cygwin-3_4-branch they currently go to release/3.4.8.
> > > For the main|master branch they currently go where?
> > 
> > release/3.5.0
> > 
> > An entry there is only necessary if it doesn't get picked for 3.4
> > anyway.
> 
> Ah, that helps me understand.
> 
> > > I hope to get it right the first time ;-).
> > 
> > Is the release model confusing?  If so, can you explain why?
> 
> I think I haven't been paying close enough attention and have been doing the
> relnote updates by rote.  But there being two active branches and I
> (understandably) don't determine which releases my commits go to means I
> should wait until they show up on the cvs-patches list, then I will know
> which relnote files to update.  That should work OK, right?
> 
> Is it preferred that relnote updates should be separate patches from the code updates?

The relnote may be a patch on its own in a series. And often is, given
that developers are not natural documenters :}

Theoretically it's preferred to be a patch on its own, because mixing
code and docs within the same patch often results in conflicts when
backporting or if it turns out that a patch has to be reverted.
Fortunately we don't have to maintain CVS Changelogs anymore...

However, if you're doing a bugfix of stuff which already *is* in the 3.4
branch (it's what others call the "stable" branch), then the idea is
that we fix it in the 3.4 branch.

For that, the patch goes into the main branch in the first place, then
it's cherry-picked into the 3.4 branch, and the relnote logically goes
into the current release file of the "not-yet-released" 3.4 version.

You can simply grep for CYGWIN_VERSION_DLL_MINOR in the file
winsup/include/cygwin/version.h in the 3.4 branch to see which release
message file is the right one at the moment.  If you do this now, you'll
get

  #define CYGWIN_VERSION_DLL_MINOR 8

Talking about conflicts.  There's always a (small!) chance that your
patch to main doesn't apply cleanly in the 3.4 branch due to other
patches already in the main branch. In that case you have to create two
versions of the patch, one for main, the other for 3.4.  Ideally the 3.4
backport contains a "Conflicts:" tag which explains why the backport
differs from it's sibling in main branch patch.  The relnote still goes
into the 3.4.x release file, though.


HTH,
Corinna

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-07-11  9:45 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-11  8:05 Where should relnote updates for Cygwin DLL patches be going? Mark Geisert
2023-07-11  8:21 ` Corinna Vinschen
2023-07-11  8:49   ` Mark Geisert
2023-07-11  9:45     ` Corinna Vinschen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).