Skip to content

Commit

Permalink
doc: make openssl maintenance position independent
Browse files Browse the repository at this point in the history
It used to have some `cd` commands that if done literally would
invalidate the subsequent commands. Modify them to be more accurate,
which also simplifies pasting them directly into the console from the
guide while doing an update.

PR-URL: nodejs#32977
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Gerhard Stöbich <[email protected]>
  • Loading branch information
sam-github authored and addaleax committed Apr 27, 2020
1 parent 2439071 commit 684a81d
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions doc/guides/maintaining-openssl.md
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ This updates all sources in deps/openssl/openssl by:
Use `make` to regenerate all platform dependent files in
`deps/openssl/config/archs/`:
```sh
% cd deps/openssl/config; make
% make -C deps/openssl/config
```

## 3. Check diffs
Expand All @@ -66,8 +66,7 @@ Check diffs if updates are right. Even if no updates in openssl
sources, `buildinf.h` files will be updated for they have a timestamp
data in them.
```sh
% cd deps/openssl/config
% git diff
% git diff -- deps/openssl
```

*Note*: On Windows, OpenSSL Configure generates `makefile` that can be
Expand Down Expand Up @@ -95,8 +94,7 @@ The commit message can be (with the openssl version set to the relevant value):
After an OpenSSL source update, all the config files need to be
regenerated and committed by:
$ cd deps/openssl/config
$ make
$ make -C deps/openssl/config
$ git add deps/openssl/config/archs
$ git add deps/openssl/openssl/include/crypto/bn_conf.h
$ git add deps/openssl/openssl/include/crypto/dso_conf.h
Expand Down

0 comments on commit 684a81d

Please sign in to comment.