Skip to content

Commit

Permalink
More fish-shell related fixes
Browse files Browse the repository at this point in the history
* Use "source" instead of "."

* Handle CDPATH in `opam config env`
  • Loading branch information
Hugo Musso Gualandi committed May 31, 2015
1 parent c87a280 commit adf191b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/client/opamConfigCommand.ml
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ let print_sexp_env env =
let print_fish_env env =
List.iter (fun (k,v) ->
match k with
| "PATH" | "MANPATH" ->
| "PATH" | "MANPATH" | "CDPATH" ->
let v = OpamStd.String.split_delim v ':' in
OpamConsole.msg "set -gx %s %s;\n" k
(OpamStd.List.concat_map " " (Printf.sprintf "%S") v)
Expand Down
2 changes: 1 addition & 1 deletion src/state/opamState.ml
Original file line number Diff line number Diff line change
Expand Up @@ -1866,7 +1866,7 @@ let source t ~shell ?(interactive_only=false) f =
| `csh ->
Printf.sprintf "source %s >& /dev/null || true\n" (file f)
| `fish ->
Printf.sprintf ". %s > /dev/null 2> /dev/null or true\n" (file f)
Printf.sprintf "source %s > /dev/null 2> /dev/null or true\n" (file f)
| _ ->
Printf.sprintf ". %s > /dev/null 2> /dev/null || true\n" (file f)
in
Expand Down

0 comments on commit adf191b

Please sign in to comment.