* Re [Cygwin PATCH */9] tzcode resync -- for discussion only
@ 2020-05-13 22:40 Mark Geisert
2020-05-19 9:48 ` Corinna Vinschen
0 siblings, 1 reply; 3+ messages in thread
From: Mark Geisert @ 2020-05-13 22:40 UTC (permalink / raw)
To: cygwin-patches
I'm not absolutely sure yet but I think this patch set isn't complete.
What's been posted is OK for discussion (on cygwin-developers?) but would
need to be augmented if you're going to apply as-is.
I had a git commit/revert mishap and these are recovered file versions. I
belatedly discovered there's no record in the patches of my creating
directory winsup/cygwin/tzcode or of my deleting localtime.cc and
tz_posixrules.h from winsup/cygwin in my local repository.
When it becomes time to submit the final patch versions, I'll do so from a
brand new repository. Sorry for any confusion.
..mark
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Re [Cygwin PATCH */9] tzcode resync -- for discussion only
2020-05-13 22:40 Re [Cygwin PATCH */9] tzcode resync -- for discussion only Mark Geisert
@ 2020-05-19 9:48 ` Corinna Vinschen
2020-05-19 10:06 ` Corinna Vinschen
0 siblings, 1 reply; 3+ messages in thread
From: Corinna Vinschen @ 2020-05-19 9:48 UTC (permalink / raw)
To: cygwin-patches
On May 13 15:40, Mark Geisert wrote:
> I'm not absolutely sure yet but I think this patch set isn't complete.
> What's been posted is OK for discussion (on cygwin-developers?) but would
> need to be augmented if you're going to apply as-is.
>
> I had a git commit/revert mishap and these are recovered file versions. I
> belatedly discovered there's no record in the patches of my creating
> directory winsup/cygwin/tzcode or of my deleting localtime.cc and
> tz_posixrules.h from winsup/cygwin in my local repository.
>
> When it becomes time to submit the final patch versions, I'll do so from a
> brand new repository. Sorry for any confusion.
Patchset discussions should stay on cygwin-patches as part of the set.
Please resend the patchset in the current state on top of current git.
Please add a cover letter (git format-patch --cover-letter ...) and just
start the discussion yourself in the cover letter. Note what you
think is missing from the patchset and what you think needs addition or
change, so we're vaguely on the same level as you :)
Thanks,
Corinna
--
Corinna Vinschen
Cygwin Maintainer
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Re [Cygwin PATCH */9] tzcode resync -- for discussion only
2020-05-19 9:48 ` Corinna Vinschen
@ 2020-05-19 10:06 ` Corinna Vinschen
0 siblings, 0 replies; 3+ messages in thread
From: Corinna Vinschen @ 2020-05-19 10:06 UTC (permalink / raw)
To: cygwin-patches
On May 19 11:48, Corinna Vinschen wrote:
> On May 13 15:40, Mark Geisert wrote:
> > I'm not absolutely sure yet but I think this patch set isn't complete.
> > What's been posted is OK for discussion (on cygwin-developers?) but would
> > need to be augmented if you're going to apply as-is.
> >
> > I had a git commit/revert mishap and these are recovered file versions. I
> > belatedly discovered there's no record in the patches of my creating
> > directory winsup/cygwin/tzcode or of my deleting localtime.cc and
> > tz_posixrules.h from winsup/cygwin in my local repository.
> >
> > When it becomes time to submit the final patch versions, I'll do so from a
> > brand new repository. Sorry for any confusion.
>
> Patchset discussions should stay on cygwin-patches as part of the set.
>
> Please resend the patchset in the current state on top of current git.
> Please add a cover letter (git format-patch --cover-letter ...) and just
> start the discussion yourself in the cover letter. Note what you
> think is missing from the patchset and what you think needs addition or
> change, so we're vaguely on the same level as you :)
Scratch that, I didn't see your v2 yet, sorry!
Corinna
--
Corinna Vinschen
Cygwin Maintainer
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-05-19 10:06 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-13 22:40 Re [Cygwin PATCH */9] tzcode resync -- for discussion only Mark Geisert
2020-05-19 9:48 ` Corinna Vinschen
2020-05-19 10:06 ` 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).