David Cantrell 🏏Engaging Smug Mode. Compare http://matrix.cpantesters.org/?dist=App-Brl2Brl+0.04 and http://matrix.cpantesters.org/?dist=App-Brl2Brl+0.05, in which I helped the author sort out his dependencies and data. #braille #accessibility #perl @Perl Alt text for links: test results for App::Brl2Brl, perl code for converting between text encoded in one Braille character set and another. Version 0.04 has lots of test failures, 0.05 passes all its tests all the time, for great portability justice. 1/3
The older version uses #liblouis's data files, but assumes liblouis is already installed. It also assumes that liblouis is installed in one particular place, so isn't very portable - it will fail at the whim of a platform's package manager or depending on whether you installed liblouis via the package manager or from source. The new version bundles the data files with the code and can always find them, so it works no matter whether you're on #Linux, a Mac, #IllumOS, or anything in between. 2/3
I realise that bundling the data isn't quite ideal, but this was a quick fix to get it working. I've added creating an #XS binding and a #PerlAlien jobby for liblouis to my to-do list, and once that's done App::Brl2Brl can switch to using that instead of bundled data. 3/3
Gene Boggs@DrHyde Bravo! +1 Open source in action. :D
Mark Gardner@DrHyde @Perl Nicely done! https://github.com/larsbjorndal/App-Brl2Brl/pull/1
How did you get in touch with Lars in the first place?
Bundle liblouis data files with the module by DrHyde · Pull Request #1 · larsbjorndal/App-Brl2Brl
As promised, this bundles the data files with the module. The module will use the bundled copy by default so you will need to periodically update them. The user can over-ride the default by either ...
@mjgardner @Perl he asked for help figuring out why it was failing in my CPAN-tester.