gEDA-cvs: CVS update: ChangeLog

[cvspsrv@xxxxxxxx]

  User: cnieves 
  Date: 07/06/09 12:09:37

  Modified:    .        ChangeLog
  Log:
  Avoid writting '\r' to files on MinGW.
  
  
  
  Applied patch #1733826 from Cesar Strauss. Thanks.
  
  When using the "w" mode in fopen, MinGW translates LF into CR-LF. 
  
  This creates havoc when reading the same file again, when sharing 
  
  schematics files with other systems, and when placing them in a VCS.
  
  
  
  This can be avoided by using the "wb" mode. The "b" is ignored on 
  
  all POSIX conforming systems, according to these sources:
  
  
  
  http://www.die.net/doc/linux/man/man3/fopen.3.html
  
  http://www.opengroup.org/onlinepubs/009695399/functions/fopen.html
  
  
  
  There are other ways, however. Adding
  
  
  
  #include <fcntl.h>
  
  int _fmode = _O_BINARY;
  
  
  
  on some source file should work as well. Another way is linking 
  
  against the MinGW supplied binmode.o, which does exactly that.
  
  
  
  
  Revision  Changes    Path
  1.697                eda/geda/gaf/gschem/ChangeLog
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: ChangeLog
  ===================================================================
  RCS file: /home/cvspsrv/cvsroot/eda/geda/gaf/gschem/ChangeLog,v
  retrieving revision 1.696
  retrieving revision 1.697
  diff -u -b -r1.696 -r1.697
  --- ChangeLog	9 Jun 2007 15:14:07 -0000	1.696
  +++ ChangeLog	9 Jun 2007 16:09:37 -0000	1.697
  @@ -21,6 +21,27 @@
   
   2007-06-09 Carlos Nieves Onega <cnieves (AT) iespana (DOT) es>
   
  +	* src/x_menus.c: Avoid writting '\r' to files on MinGW.
  +
  +	Applied patch #1733826 from Cesar Strauss. Thanks.
  +	When using the "w" mode in fopen, MinGW translates LF into CR-LF. 
  +	This creates havoc when reading the same file again, when sharing 
  +	schematics files with other systems, and when placing them in a VCS.
  +
  +	This can be avoided by using the "wb" mode. The "b" is ignored on 
  +	all POSIX conforming systems, according to these sources:
  +
  +	http://www.die.net/doc/linux/man/man3/fopen.3.html
  +	http://www.opengroup.org/onlinepubs/009695399/functions/fopen.html
  +
  +	There are other ways, however. Adding
  +
  +	#include <fcntl.h>
  +	int _fmode = _O_BINARY;
  +
  +	on some source file should work as well. Another way is linking 
  +	against the MinGW supplied binmode.o, which does exactly that.
  +
   	* src/x_image.c: Added EPS support.
   
   	Applied patch #1713747 (with minor tweaks) from Wojciech Kazubski 
  
  
  


_______________________________________________
geda-cvs mailing list
geda-cvs@xxxxxxxxxxxxxx
http://www.seul.org/cgi-bin/mailman/listinfo/geda-cvs