Skip to content

Create an example of using with leptonica's c-api#60

Open
ccouzens wants to merge 4 commits intomasterfrom
c-api-fallback-support
Open

Create an example of using with leptonica's c-api#60
ccouzens wants to merge 4 commits intomasterfrom
c-api-fallback-support

Conversation

@ccouzens
Copy link
Copy Markdown
Collaborator

It needed support to be able to get the mut pointer from leptonica-plumbing

ccouzens/leptonica-plumbing@4a721e7

And for leptonica-plumbing pix to self reference

ccouzens/leptonica-plumbing@545708c

It uses this so that methods can take AsRefLeptonicaPlumbing::Pix which both leptess and leptonica-plumbing satisfy.

Addresses #59

It needed support to be able to get the mut pointer from leptonica-plumbing

ccouzens/leptonica-plumbing@4a721e7

And for leptonica-plumbing pix to self reference

ccouzens/leptonica-plumbing@545708c

It uses this so that methods can take AsRef<LeptonicaPlumbing::Pix> which
both leptess and leptonica-plumbing satisfy.

Addresses #59
```
5 | use self::tesseract_sys::TessBaseAPIInit5;
  |     ^^^^^^^^^^^^^^^^^^^^^----------------
  |     |                    |
  |     |                    help: a similar name exists in the module: `TessBaseAPIInit1`
  |     no `TessBaseAPIInit5` in the root
```

https://github.com/houqp/leptess/actions/runs/7589902627/job/20675452376

This will be because the environment github uses doesn't have the new version of tesseract.
- uses: actions/checkout@v2
- name: Build
run: cargo build --verbose
run: cargo build --verbose --no-default-features
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Fix build failures that depend on a newer version of tesseract

https://github.com/houqp/leptess/actions/runs/7589902627/job/20675452376

So that we can disable tesseract 5-2 feature
tesseract-plumbing = "~0.11"
tesseract-plumbing = {version = "~0.11", default-features = false}
thiserror = "1"
leptonica-plumbing = "~1.3.0"
Copy link
Copy Markdown
Collaborator Author

@ccouzens ccouzens Jan 19, 2024

Choose a reason for hiding this comment

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

Specify this directly rather than as a transient dependency, as the example depends on the latest version.

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