On Oct 2, 2014, at 11:17 AM, Ralph Castain <r...@open-mpi.org>
 wrote:

> 
> On Oct 2, 2014, at 9:13 AM, Dave Goodell (dgoodell) <dgood...@cisco.com> 
> wrote:
> 
>> On Oct 2, 2014, at 10:38 AM, git...@crest.iu.edu wrote:
>> 
>>> This is an automated email from the git hooks/post-receive script. It was
>>> generated because a ref change was pushed to the repository containing
>>> the project "open-mpi/ompi".
>>> 
>>> The branch, master has been updated
>>>      via  3263f721b6a21966a1c1eea0fdac2a558a15db06 (commit)
>>>     from  f21c349bcb3f7c322805d505484951642d1c7965 (commit)
>>> 
>>> Those revisions listed above that are new to this repository have
>>> not appeared on any other notification email; so we list those
>>> revisions in full, below.
>>> 
>>> - Log -----------------------------------------------------------------
>>> https://github.com/open-mpi/ompi/commit/3263f721b6a21966a1c1eea0fdac2a558a15db06
>>> 
>>> commit 3263f721b6a21966a1c1eea0fdac2a558a15db06
>>> Author: Ralph Castain <r...@open-mpi.org>
>>> Date:   Thu Oct 2 08:37:18 2014 -0700
>>> 
>>>   Strip crlf line endings
>>> 
>>> diff --git a/ompi/contrib/vt/vt/extlib/otf/otf_vc08.sln 
>>> b/ompi/contrib/vt/vt/extlib/otf/otf_vc08.sln
>>> index 38498d6..7e887fc 100644
>>> --- a/ompi/contrib/vt/vt/extlib/otf/otf_vc08.sln
>>> +++ b/ompi/contrib/vt/vt/extlib/otf/otf_vc08.sln
>> 
>> Isn't this file a Windows-specific file that probably should still have CRLF 
>> line endings?
> 
> No idea, but we don't support Windows any more, so who cares?

Right, but when the VT folks slurp in a new update from their upstream, won't 
this just get clobbered back to CRLF?  Or worse, cause some sort of merge 
conflict for them?

I don't especially care, just thought it was odd to remove the CRLFs from a 
Windows-only file.

-Dave

Reply via email to