Skip to content

Conversation

@james-knippes
Copy link

Maintainer: @micmac1
Compile tested: lantiq/xway (openwrt-24.10) , lantiq/xrx200 (openwrt-24.10, main)
Run tested: lantiq/xway (openwrt-24.10) , lantiq/xrx200 (openwrt-24.10)

Description: These options allow compiling asterisk with or without crypto and ssl support. This is mainly useful for image size optimization for small devices.

These options allow compiling asterisk with or without crypto
and ssl support. This is mainly useful for image size optimization
for small devices.

Signed-off-by: Jan Knittel <knittel@mailbox.org>
@james-knippes
Copy link
Author

Would this qualify for being backported to 24.10 ? I'm not sure if we could bend the rules and qualify this as 'fix' as this essentially fixes building asterisk for devices with small flash sizes.

@james-knippes
Copy link
Author

I just realized that @jslachta is listed as maintainer of the asterisk package. I don't know where i got the micmac1 from and micmac1 seems to be inactive for a while anyways. Maybe you could take a look at this Jiri ?

@Ansuel
Copy link
Member

Ansuel commented Nov 22, 2025

@james-knippes i wonder if it can be compiled with only crypto... is it even possible? Doesn't that also require libopenssl (for the libcrypto?)

@james-knippes
Copy link
Author

@Ansuel I'm not sure if it does make sense to only use --with-crypto. I only tried building with both off or both on. But the flags are exposed as separate flags in the configure step and it seems like there are some checks in the build to make sure the config is valid.
So I think we should be fine exposing the flags separately just like the asterisk build does?

@Ansuel
Copy link
Member

Ansuel commented Nov 24, 2025

@james-knippes please test with one of them off (the SSL one) this might be ok for normal installation where libcrypto might be externally provided but on openwrt both library are from openssl so we need openssl dependency for both.

We can keep both config but we need to make sure the dependency is consistent

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.

2 participants