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

Improve snap packaging #1892

Merged
merged 7 commits into from Sep 3, 2018

Conversation

brlin-tw
Copy link
Contributor

@brlin-tw brlin-tw commented Sep 3, 2018

This PR fixes some issues regarding the snap packaging.

Conforming-to: tesseract/CONTRIBUTING.md at master · tesseract-ocr/tesseract https://github.com/tesseract-ocr/tesseract/blob/master/CONTRIBUTING.md
Signed-off-by: 林博仁(Buo-ren, Lin) Buo.Ren.Lin@gmail.com

Snapcraft supports automatic `git describe` based versioning when the
`version` key is set to `git`.  This gives more information of the
source status of a snap distribution.

Refer-to: Snapcraft.yaml reference - doc - snapcraft.io
<https://forum.snapcraft.io/t/snapcraft-yaml-reference/4276>
Signed-off-by: 林博仁(Buo-ren, Lin) <Buo.Ren.Lin@gmail.com>
This change allows downstream packagers to publish the snap to the
candidate and stable channels.

Signed-off-by: 林博仁(Buo-ren, Lin) <Buo.Ren.Lin@gmail.com>
This allows cleaner diffs for future modifications using the YAML's
"folded" block scalar style.

Refer-to: Folded Style - Block Scalar Styles - Block Styles
<http://yaml.org/spec/1.2/spec.html#id2796251>
Signed-off-by: 林博仁(Buo-ren, Lin) <Buo.Ren.Lin@gmail.com>
This allows access to the files under `/media/*` and `/run/media/*`, as
the connection is not automatic connect instructions is also added to
the snap's description.

Refer-to: Interfaces - doc - snapcraft.io
<https://forum.snapcraft.io/t/interfaces/6154>
Signed-off-by: 林博仁(Buo-ren, Lin) <Buo.Ren.Lin@gmail.com>
Signed-off-by: 林博仁(Buo-ren, Lin) <Buo.Ren.Lin@gmail.com>
Snapcraft will soonly drop the feature to automatic staging the
depending libraries into the snap, specify it manually to avoid breakage
in the future.

This deals with the following warning message in the build:

```
Priming leptonica
Files from the build host were migrated into the snap to satisfy
dependencies that would otherwise not be met. This feature will be
removed in a future release. If these libraries are needed in the final
snap, ensure that the following are either satisfied by a stage-packages
entry or through a part:
usr/lib/x86_64-linux-gnu/libjbig.so.0
usr/lib/x86_64-linux-gnu/libjpeg.so.8
usr/lib/x86_64-linux-gnu/libopenjp2.so.7
usr/lib/x86_64-linux-gnu/libtiff.so.5
Priming tesseract
Files from the build host were migrated into the snap to satisfy
dependencies that would otherwise not be met. This feature will be
removed in a future release. If these libraries are needed in the final
snap, ensure that the following are either satisfied by a stage-packages
entry or through a part:
usr/lib/x86_64-linux-gnu/libgomp.so.1
```

Signed-off-by: 林博仁(Buo-ren, Lin) <Buo.Ren.Lin@gmail.com>
Signed-off-by: 林博仁(Buo-ren, Lin) <Buo.Ren.Lin@gmail.com>
@brlin-tw
Copy link
Contributor Author

brlin-tw commented Sep 3, 2018

The c-i build result doesn't seem to be the result of this PR, just saying.

@egorpugin egorpugin merged commit 3a7cd2b into tesseract-ocr:master Sep 3, 2018
@brlin-tw brlin-tw deleted the improve-snap-packaging branch August 25, 2019 07:06
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.

None yet

2 participants