This is the mail archive of the
insight@sourceware.org
mailing list for the Insight project.
Re: [PATCH] Support Cygwin network paths in tcl
On Mon, Aug 07, 2006 at 10:54:50AM -0700, Keith Seitz wrote:
>Thiemo Seufer wrote:
>>>Is this in the upstream Tcl distro?
>>Likely not.
>>
>>>I'm not really sure how I feel about maintaining a branch. I'm still
>>>trying (albeit feebly) to whack tcl et al from the repository
>>>altogether.
>>
>>It might make sense to add a reference to upstream in MAINTAINERS then.
>
>www.tcl.tk for more tcl information.
>
>Alas, last I checked, cygwin was a deprecated platform for Tcl due to
>the lack of a maintainer. I don't know if this has changed, but I
>guess I shouldn't take that risk.
>
>That means that if insight is to continue to run on windows, I would
>need to maintain patches to enable cygwin on tcl or maintain our
>current copy for the benefit of cygwin users. I don't particularly
>like either solution, but I don't see that I have a choice unless a
>cygwin Tcl maintainer materializes.
>
>So given those constraints, I don't see a problem with the patch. I'll
>wait a few days, and if there aren't any unsurmountable objections,
>I'll commit it.
My only non-insurmountable objection is that these aren't "cygwin network
paths", they are "windows network paths". I think that any comnents or
ChangeLogs should be modified to reflect that fact.
cgf