365691 - Useful error message and conversion tool when switch from textmode (cygwin) to binmode (MSYS)

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Benjamin Smedberg]

The patch will provide a useful error message when users switch from a textmode source tree (cygwin default) to a binmode tree (MSYS default).

Comment 1

[Benjamin Smedberg]

Attachment: detect problems in client.mk, and add converter script, rev. 1

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 250201 [details] [diff] [review]
detect problems in client.mk, and add converter script, rev. 1

r=dbaron, although I can't claim to be much of a python expert, and the perlfunc definition of |binmode| doesn't make much sense to me.

One concern:  should your python script check for the conditions that make it do dos2unix rather than some other behavior?  All it really does is read in as text and write out as binary.

Comment 3

[Benjamin Smedberg]

Since dos2unix is repeatable, I don't think there's any harm in doing it whenever we encounter a non-textmode file.

Fixed on trunk.

Comment 4

[[e-mail]]

Comment on attachment 250201 [details] [diff] [review]
detect problems in client.mk, and add converter script, rev. 1

>+print """This command will convert the source tree at
>+%s 
>+to an MSYS-compatible (unix mode) source tree. You can run this
>+command multiple times safely. Are you sure you want to continue (Y/N)? """ % topsrcdir,
Unfortunately you can't run this command even once safely. Excluding binary files, we have 340 files in the tree containing \r\n characters. Of those, 71 either have mixed line endings or should really be binary.

Comment 5

[[e-mail]]

Attachment: list of suspect files

Note: this is based on MOZ_CO_MODULES=suite so there may be others.