Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use src/distrooptions.h as another source #274

Merged
merged 1 commit into from
Feb 13, 2024
Merged

Use src/distrooptions.h as another source #274

merged 1 commit into from
Feb 13, 2024

Conversation

mkj
Copy link
Owner

@mkj mkj commented Jan 22, 2024

This makes it easier for distributions to patch the source directory, rather than needing to create localoptions.h in the build directory.

Makefile's OPTION_HEADERS is removed, the wildcard *.h covers it

This makes it easier for distributions to patch the source directory,
rather than needing to create localoptions.h in the build directory.

Makefile's OPTION_HEADERS is removed, the wildcard *.h covers it
@mkj mkj merged commit 454591d into master Feb 13, 2024
16 checks passed
@mkj mkj deleted the distrooptions branch February 13, 2024 13:32
geissonator pushed a commit to openbmc/openbmc that referenced this pull request Feb 20, 2024
Per the dropbear documentation, the localoptions.h file needs to be
copied to its build directory to take effect.

This file can later be renamed to src/distrooptions.h once the following
dropbear change is merged:
mkj/dropbear#274

Tested:

Before change:
$ ssh -vv p10bmc
...
debug2: MACs ctos: hmac-sha1,hmac-sha2-256
debug2: MACs stoc: hmac-sha1,hmac-sha2-256

After change:
$ ssh -vv p10bmc
...
debug2: MACs ctos: hmac-sha1,hmac-sha2-256,hmac-sha2-512
debug2: MACs stoc: hmac-sha1,hmac-sha2-256,hmac-sha2-512

Change-Id: I1a4b108d1a87831f4623c7e3e1e2beb0b8655441
Signed-off-by: Adriana Kobylak <[email protected]>
rfrandse pushed a commit to ibm-openbmc/openbmc that referenced this pull request Mar 28, 2024
Per the dropbear documentation, the localoptions.h file needs to be
copied to its build directory to take effect.

This file can later be renamed to src/distrooptions.h once the following
dropbear change is merged:
mkj/dropbear#274

Tested:

Before change:
$ ssh -vv p10bmc
...
debug2: MACs ctos: hmac-sha1,hmac-sha2-256
debug2: MACs stoc: hmac-sha1,hmac-sha2-256

After change:
$ ssh -vv p10bmc
...
debug2: MACs ctos: hmac-sha1,hmac-sha2-256,hmac-sha2-512
debug2: MACs stoc: hmac-sha1,hmac-sha2-256,hmac-sha2-512

Change-Id: I1a4b108d1a87831f4623c7e3e1e2beb0b8655441
Signed-off-by: Adriana Kobylak <[email protected]>
rfrandse pushed a commit to ibm-openbmc/openbmc that referenced this pull request Apr 19, 2024
Per the dropbear documentation, the localoptions.h file needs to be
copied to its build directory to take effect.

This file can later be renamed to src/distrooptions.h once the following
dropbear change is merged:
mkj/dropbear#274

Tested:

Before change:
$ ssh -vv p10bmc
...
debug2: MACs ctos: hmac-sha1,hmac-sha2-256
debug2: MACs stoc: hmac-sha1,hmac-sha2-256

After change:
$ ssh -vv p10bmc
...
debug2: MACs ctos: hmac-sha1,hmac-sha2-256,hmac-sha2-512
debug2: MACs stoc: hmac-sha1,hmac-sha2-256,hmac-sha2-512

Change-Id: I1a4b108d1a87831f4623c7e3e1e2beb0b8655441
Signed-off-by: Adriana Kobylak <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant