Skip to content

Commit

Permalink
Merge pull request #4449 from Blaisorblade/openssl-fallback
Browse files Browse the repository at this point in the history
openssl invocation: Fix fallback (part of #4448)
  • Loading branch information
rjbou authored Nov 26, 2020
2 parents 51c160b + afa1622 commit e93dd4a
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 1 deletion.
1 change: 1 addition & 0 deletions master_changes.md
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,7 @@ New option/command/subcommand are prefixed with ◈.
* Initialise random [#4391 @rjbou]
* Fix CLI debug log printed without taking into account debug sections [#4391 @rjbou]
* Internal caches: use size checks from Marshal [#4430 @AltGr]
* openssl invocation: Fix permission denied fallback [#4449 @Blaisorblade - fix #4448]

## Test
* Ensure that a cold `dune runtest` works [#4375 @emillon]
Expand Down
2 changes: 1 addition & 1 deletion src/core/opamHash.ml
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ let compute ?(kind=default_kind) file = match kind with
| _ ->
log "openssl error, use internal sha library";
OpamSHA.hash kind file
with OpamSystem.Command_not_found _ | OpamSystem.Process_error _ ->
with OpamSystem.Command_not_found _ | OpamSystem.Process_error _ | OpamSystem.Permission_denied _ ->
log "openssl not found, use internal sha library";
OpamSHA.hash kind file
in
Expand Down
2 changes: 2 additions & 0 deletions src/core/opamSystem.mli
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@ exception Process_error of OpamProcess.result

exception Command_not_found of string

exception Permission_denied of string

(** raise [Process_error] *)
val process_error: OpamProcess.result -> 'a

Expand Down

0 comments on commit e93dd4a

Please sign in to comment.