Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0004175OCamlOCaml generalpublic2006-11-28 14:292014-08-18 20:21
Reporteroandrieu 
Assigned To 
PrioritynormalSeverityminorReproducibilityalways
StatusconfirmedResolutionopen 
PlatformOSOS Version
Product Version 
Target Version4.02.1+devFixed in Version 
Summary0004175: runtime header file io.h conflicts with a system header file
DescriptionLe fichier byterun/io.h est en conflit avec le io.h standard de Windows. La règle de compilation de win32.o dans Makefile.nt essaye bien de bricoler pour contourner le problème mais ce n'est pas suffisant : par exemple sous mingw, <fcntl.h> fait un include <io.h> mais lors de la compilation dans amsrun/ c'est le io.h de caml qui est inclus (à cause de l'option -I../byterun).

Il faudrait soit:
- renommer io.h en caml_io.h par exemple
- éviter complètement l'option -I../byterun dans asmrun/Makefile et asmrun/Makefile.nt en liant/copiant les fichiers .h de la même façon que les fichiers .c

En attachement, un patch avec quelques #include supplémentaires pour éviter des warnings du compilateur C.
Tagspatch
Attached Filespatch file icon ocaml-asmrun_win.patch [^] (1,024 bytes) 2006-11-28 14:29 [Show Content]

- Relationships
duplicate of 0005887confirmed Name clash with memory.h header file with the Android NDK 

-  Notes
(0007749)
vouillon (reporter)
2012-07-13 11:24

There are several warnings (implicit declaration of function) when compiling win32unix with MinGW because of this issue. Hopefully, they are harmless...
(0008745)
Camarade_Tux (reporter)
2013-01-13 12:18

I've had issues with this which have prevented me from building (currently adding cross-compilation support to ocaml). Even if we only consider the warnings, they don't seem completely harmless. As far as I'm concerned, I've renamed the header to "_caml_io.h".

I don't understand why '#include <io.h>' uses the caml header: using '<io.h>' instead of '"io.h"' is supposed to pull the system header... It's not the first time I notice something like that however.
(0009677)
doligez (administrator)
2013-07-02 16:55

This PR is a kind of subset of 5887, let's continue the discussion in that PR instead of here.

- Issue History
Date Modified Username Field Change
2006-11-28 14:29 oandrieu New Issue
2006-11-28 14:29 oandrieu File Added: ocaml-asmrun_win.patch
2007-11-10 13:25 xleroy Status new => acknowledged
2012-07-11 16:13 doligez Target Version => 4.01.0+dev
2012-07-13 11:24 vouillon Note Added: 0007749
2012-07-31 13:37 doligez Target Version 4.01.0+dev => 4.00.1+dev
2012-09-14 23:51 doligez Target Version 4.00.1+dev => 4.00.2+dev
2013-01-13 12:18 Camarade_Tux Note Added: 0008745
2013-07-02 16:54 doligez Relationship added duplicate of 0005887
2013-07-02 16:55 doligez Note Added: 0009677
2013-07-02 16:55 doligez Target Version 4.00.2+dev => 4.01.0+dev
2013-07-29 14:36 doligez Target Version 4.01.0+dev => 4.02.0+dev
2013-09-03 11:27 doligez Status acknowledged => confirmed
2013-09-04 17:28 doligez Tag Attached: patch
2014-08-18 20:21 doligez Target Version 4.02.0+dev => 4.02.1+dev


Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker