Skip to content

Commit

Permalink
Full OS name for all BSD variants
Browse files Browse the repository at this point in the history
  • Loading branch information
@samoht
samoht committed Mar 14, 2013
1 parent e37c9a9 commit abe385c
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/core/opamGlobals.ml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -146,10 +146,10 @@ let os () =
let string_of_os = function
| Darwin -> "darwin"
| Linux -> "linux"
| FreeBSD
| OpenBSD
| NetBSD
| DragonFly -> "bsd"
| FreeBSD -> "freebs"

This comment has been minimized.

Sign in to view

Copy link
@avsm

avsm Mar 14, 2013

Member

woops... "freebs" -> "freebsd"
| OpenBSD -> "openbsd"
| NetBSD -> "netbsd"
| DragonFly -> "dragonfly"
| Cygwin -> "cygwin"
| Win32 -> "win32"
| Unix -> "unix"
Expand Down

3 comments on commit abe385c

@samoht
Copy link
Member Author

@samoht samoht commented on abe385c Mar 14, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

woops ...

@tuong
Copy link

@tuong tuong commented on abe385c Mar 16, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

woops... https://github.com/OCamlPro/opam-repository/blob/master/packages/camlidl.1.05/opam

@samoht
Copy link
Member Author

@samoht samoht commented on abe385c Mar 16, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for pointing this out, we need to fix the compilation on BSD for some remaining packages anyway (and that's way we needed to distinguish between the different BSD system).

Please sign in to comment.