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

fbreader: 0.99.4 -> 0.99.6, switch to fetchFromGitHub, add more build options #45840

Merged
merged 1 commit into from Sep 2, 2018

Conversation

oxij
Copy link
Member

@oxij oxij commented Aug 31, 2018

This, apparently, is not an official release (it's tagged, but the tarball is not available on the official site), but this repo is the official repo mentioned on the official site.

This fixes a bunch of very annoying bugs present 0.99.4, e.g.

$ FBReader filename

does not crash anymore. Yay!

Motivation for this change

I hate that crash.

Things done
  • Build and works for me on Linux.
  • I added the plumbing for Darwin emulating other similar packages, but have not tested it.

… options

This, apparently, is not an official release (it's tagged, but the tarball is not
available on the official site), but this repo is the official repo mentioned on
the official site.

This fixes a bunch of very annoying bugs present in 0.99.4, e.g.

```
$ FBReader filename
```

does not crash anymore. Yay!
@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: fbreader

Partial log (click to expand)

shrinking RPATHs of ELF executables and libraries in /nix/store/69jyqgan9rvsv28mpqfxlv4p38fhh6gg-fbreader-qt4-0.99.6
shrinking /nix/store/69jyqgan9rvsv28mpqfxlv4p38fhh6gg-fbreader-qt4-0.99.6/bin/FBReader
shrinking /nix/store/69jyqgan9rvsv28mpqfxlv4p38fhh6gg-fbreader-qt4-0.99.6/lib/libzltext.so.0.14.0
shrinking /nix/store/69jyqgan9rvsv28mpqfxlv4p38fhh6gg-fbreader-qt4-0.99.6/lib/libzlcore.so.0.14.0
shrinking /nix/store/69jyqgan9rvsv28mpqfxlv4p38fhh6gg-fbreader-qt4-0.99.6/lib/zlibrary/ui/zlui-qt4.so
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/69jyqgan9rvsv28mpqfxlv4p38fhh6gg-fbreader-qt4-0.99.6/lib  /nix/store/69jyqgan9rvsv28mpqfxlv4p38fhh6gg-fbreader-qt4-0.99.6/bin
patching script interpreter paths in /nix/store/69jyqgan9rvsv28mpqfxlv4p38fhh6gg-fbreader-qt4-0.99.6
checking for references to /build in /nix/store/69jyqgan9rvsv28mpqfxlv4p38fhh6gg-fbreader-qt4-0.99.6...
/nix/store/69jyqgan9rvsv28mpqfxlv4p38fhh6gg-fbreader-qt4-0.99.6

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: fbreader

Partial log (click to expand)

shrinking RPATHs of ELF executables and libraries in /nix/store/9kwp5q2kmzn1y0kbs551pf7xsldwnmyb-fbreader-qt4-0.99.6
shrinking /nix/store/9kwp5q2kmzn1y0kbs551pf7xsldwnmyb-fbreader-qt4-0.99.6/bin/FBReader
shrinking /nix/store/9kwp5q2kmzn1y0kbs551pf7xsldwnmyb-fbreader-qt4-0.99.6/lib/zlibrary/ui/zlui-qt4.so
shrinking /nix/store/9kwp5q2kmzn1y0kbs551pf7xsldwnmyb-fbreader-qt4-0.99.6/lib/libzltext.so.0.14.0
shrinking /nix/store/9kwp5q2kmzn1y0kbs551pf7xsldwnmyb-fbreader-qt4-0.99.6/lib/libzlcore.so.0.14.0
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/9kwp5q2kmzn1y0kbs551pf7xsldwnmyb-fbreader-qt4-0.99.6/lib  /nix/store/9kwp5q2kmzn1y0kbs551pf7xsldwnmyb-fbreader-qt4-0.99.6/bin
patching script interpreter paths in /nix/store/9kwp5q2kmzn1y0kbs551pf7xsldwnmyb-fbreader-qt4-0.99.6
checking for references to /build in /nix/store/9kwp5q2kmzn1y0kbs551pf7xsldwnmyb-fbreader-qt4-0.99.6...
/nix/store/9kwp5q2kmzn1y0kbs551pf7xsldwnmyb-fbreader-qt4-0.99.6

@xeji
Copy link
Contributor

xeji commented Sep 2, 2018

a big leap from 2011 -> 2016 😄

@xeji xeji merged commit a402c8f into NixOS:master Sep 2, 2018
@obadz
Copy link
Contributor

obadz commented Dec 1, 2018

@xeji, fbreader won't build on more recent curl (due to #49514) :(

geometer/FBReader#310
https://hydra.nixos.org/jobset/nixos/trunk-combined#tabs-jobs

@vcunat
Copy link
Member

vcunat commented Dec 9, 2018

It does now; see 1970a4c.

@obadz
Copy link
Contributor

obadz commented Dec 15, 2018

Thanks a lot @vcunat.

@vvs- vvs- mentioned this pull request Jul 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants