Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0005991OCamlOCaml internal build/install (Makefiles, configure)public2013-04-18 19:452014-05-25 20:20
ReporterPedroBurito 
Assigned To 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusacknowledgedResolutionopen 
Platformarmv5tejlOSLinuxOS Version2.6.24.4
Product Version4.00.1 
Target Version4.02.0+devFixed in Version 
Summary0005991: Syntax error in tools/Makefile.shared
DescriptionI tried building ocaml from source and i got stuck with an error. i think there's missing a \ in on of the sed expressions which generates the opnames.ml file.
i provided a diff -u in the additional info section which hopefully resolves the issue.
gnu sed 4.2.2
Additional Information--- Makefile.shared 2013-04-18 19:33:55.000000000 +0200
+++ Makefile.shared.bugged 2013-04-18 19:39:29.000000000 +0200
@@ -254,7 +254,7 @@
        unset LC_COLLATE LANG || : ; \
        sed -e '/\/\*/d' \
            -e '/^#/d' \
- -e 's/enum \(.*\) {/let names_of_\1 = \[|/' \
+ -e 's/enum \(.*\) {/let names_of_\1 = [|/' \
            -e 's/};$$/ |]/' \
            -e 's/\([A-Z][A-Z_0-9a-z]*\)/"\1"/g' \
            -e 's/,/;/g' \
TagsNo tags attached.
Attached Files

- Relationships

-  Notes
(0009229)
frisch (developer)
2013-04-29 16:49

What's the error? At least with gnu sed 4.2.1, it seems to work fine.
(0009230)
PedroBurito (reporter)
2013-04-29 16:54

Sorry, i got the diff the wrong way around.
The missing \ before the [ after /let names_of_\1 = ..... made my sed fail and the makefile therefore created an empty file instead of the command listing or whatever the file actually is :)
(0009560)
doligez (administrator)
2013-06-19 12:44

Are you sure this is really the problem? I have a hard time believing that GNU sed would introduce such a big change in a bug-fix release (between 4.2.1 and 4.2.2). I also find it strange that 4.2.2 was released, but the on-line documentation on the GNU web site is still for 4.2.1.

Moreover, the man page for a "normal" sed (Mac OS X) doesn't seem to say that arbitrary characters can be escaped in the replacement string, so this is not supposed to work. It does work on MacOS, but what about the other BSDs, for example?

I don't think we want to risk including this fix before the 4.01.0 release.

- Issue History
Date Modified Username Field Change
2013-04-18 19:45 PedroBurito New Issue
2013-04-29 16:49 frisch Note Added: 0009229
2013-04-29 16:54 PedroBurito Note Added: 0009230
2013-06-19 12:44 doligez Note Added: 0009560
2013-06-19 12:44 doligez Target Version => 4.02.0+dev
2013-06-28 18:25 doligez Status new => acknowledged
2013-07-12 18:15 doligez Target Version 4.02.0+dev => 4.01.1+dev
2014-05-25 20:20 doligez Target Version 4.01.1+dev => 4.02.0+dev


Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker