Skip to content

Conversation

@juanro49
Copy link
Contributor

@juanro49 juanro49 commented Dec 18, 2025

These changes correct the compilation of the gbinder driver by linking the precompiled euicc and lpac-utils libraries included in the repository.

Copy link
Contributor

@CoelacanthusHex CoelacanthusHex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rebase onto main branch.

Copy link
Contributor

@septs septs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

REJECT, please remove rpm directory

if needs, please add a packaging information to docs/LINUX-DIST.md file

@juanro49
Copy link
Contributor Author

Hi! I have resolved the conflicts and removed the spec from the PR branch (I uploaded it without realizing that the PR was still active, so I moved it to another branch so that it would not affect the PR).

Copy link
Contributor

@septs septs Dec 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should assume your users know how to use it and only provide KEY INFORMATION.

@septs
Copy link
Contributor

septs commented Dec 22, 2025

please split as three pull requests

  • fix: gbinder driver compile fails
  • fix: dynamic drvier detection for SailfishOS
  • docs: add SailfishOS into LINUX-DIST.md file

@juanro49
Copy link
Contributor Author

please split as three pull requests

* `fix: gbinder driver compile fails`

* `fix: dynamic drvier detection for SailfishOS`

* `docs: add SailfishOS into LINUX-DIST.md file`

Thanks, I will split this in two PR, because "fix: dynamic drvier detection for SailfishOS" is solved in #401

@juanro49 juanro49 changed the title fix gbinder driver compile and dynamic drivers detection for SailfishOS fix gbinder driver compile Dec 22, 2025
@septs septs changed the title fix gbinder driver compile fix(driver): gbinder driver compile Dec 22, 2025
Copy link
Contributor

@septs septs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait for @CoelacanthusHex

Copy link
Contributor

@CoelacanthusHex CoelacanthusHex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@CoelacanthusHex CoelacanthusHex merged commit 31d5966 into estkme-group:main Dec 22, 2025
13 checks passed
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