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

optimize gettext / glib builds #1690

Merged

Conversation

benoit-pierre
Copy link
Contributor

@benoit-pierre benoit-pierre commented Nov 1, 2023

  • compile only the part of gettext we need
  • compile only a static version of libglib-2.0, since that's all sdcv needs

This change is Reviewable

@benoit-pierre benoit-pierre marked this pull request as draft November 1, 2023 15:43
@Frenzie
Copy link
Member

Frenzie commented Nov 1, 2023

As an aside and I don't know if it matters, but isn't the glib becoming a bit old?

@benoit-pierre
Copy link
Contributor Author

It's not too fresh, yeah.

@NiLuJe
Copy link
Member

NiLuJe commented Nov 1, 2023

Yeah, newer releases switched to meson and I didn't have the will to deal with it for a platform idgaf about ;).

@Frenzie
Copy link
Member

Frenzie commented Nov 1, 2023

Right, I knew I'd also skipped over it on purpose in the past but I didn't remember why.

@benoit-pierre benoit-pierre force-pushed the pr/minimal_glib_static_build branch 4 times, most recently from e1dd0e2 to 2a30afd Compare November 1, 2023 20:41
- compile only the part of gettext we need
- compile only a static version of `libglib-2.0`, since that's all sdcv needs
@benoit-pierre benoit-pierre force-pushed the pr/minimal_glib_static_build branch from 2a30afd to 3ed7ab2 Compare November 2, 2023 20:49
@Frenzie
Copy link
Member

Frenzie commented Nov 2, 2023

@benoit-pierre What does the draft status currently indicate? ^_^

@benoit-pierre
Copy link
Contributor Author

I was just waiting for #1689 to be merged.

@benoit-pierre benoit-pierre marked this pull request as ready for review November 2, 2023 21:06
@Frenzie Frenzie merged commit d191057 into koreader:master Nov 2, 2023
1 of 2 checks passed
@benoit-pierre benoit-pierre deleted the pr/minimal_glib_static_build branch November 2, 2023 21:18
Frenzie pushed a commit to koreader/koreader that referenced this pull request Nov 9, 2023
* Bump base

includes:

koreader/koreader-base#1691
koreader/koreader-base#1692
koreader/koreader-base#1689
koreader/koreader-base#1690
koreader/koreader-base#1693

* Integrate decoding of SSIDs within wpa_supplicant

The UTF-8 decoding of SSIDs is specific to wpa_supplicant. In this
patch, we move all of this decoding logic to the wpa_supplicant module.
We expose the raw bytes of the SSID to the NetworkMgr code, and make
sure to always fix bad UTF-8 before we display the SSID to the user.

Within the wpa_supplicant module, we replace the call to the
wpa_passphrase binary to get the PSK with a direct function call to
OpenSSL. This allows us to calculate the PSK over any arbitrary bytes,
including UTF-8. In the same vein, we use the hex-encoded SSID to
communicate with wpa_supplicant when setting up the network to support
arbitrary bytes in the SSID.

Unfortunately, we also remove the tests, as there is no way to unit test
local functions.
@Frenzie Frenzie mentioned this pull request Mar 26, 2024
2 tasks
benoit-pierre added a commit to benoit-pierre/koreader-base that referenced this pull request Mar 27, 2024
Not needed on other platforms, fix regression introduced in koreader#1690.
benoit-pierre added a commit to benoit-pierre/koreader-base that referenced this pull request Mar 27, 2024
Not needed on other platforms, fix regression introduced in koreader#1690.
Frenzie pushed a commit that referenced this pull request Mar 31, 2024
Not needed on other platforms, fix regression introduced in #1690.
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.

3 participants