<div dir="ltr">1) Done in next submission.<div>2) Done in next submission.</div><div><br></div><div>--Larry</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Nov 30, 2013 at 12:15 PM, King Beowulf <span dir="ltr"><<a href="mailto:kingbeowulf@gmail.com" target="_blank">kingbeowulf@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
I've tested calbire-1.13.0 with the current SBo slackbuild script and<br>
everything appears A-OK.  Recommendations:<br>
<br>
1. This line does not seems to be needed as the version string appears<br>
to be correct:<br>
# Make sure version string is correct.<br>
sed -i "/^numeric_version/s|(.*)|(${VERSION//\./, })|"<br>
src/calibre/constants.py<br>
<br>
It doesn't hurt anything, but sends my minimalist sensibilities all<br>
a-flutter...<br>
<br>
2. For "$PRGNAM-gui.desktop" (ie calibre-gui.desktop) I suggest we<br>
capitalize the menu file name:<br>
<br>
Name=calibre<br>
<br>
to<br>
<br>
Name=Calibre<br>
<br>
Some DE menus will then sort the name name "correctly."  For example,<br>
Xfce sorts capitals before lower case letters (If anyone knows how to<br>
change that I am "all ears").<br>
<br>
- -Ed<br>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v2.0.22 (GNU/Linux)<br>
<br>
iEYEARECAAYFAlKaR08ACgkQXvwMaW61dLevFQCfZW+xPKJgeYBx58CZUcO9rZ8n<br>
eTwAmQE0d7re6sR9s7AoCNf7/NI3ayAy<br>
=wPOe<br>
-----END PGP SIGNATURE-----<br>
_______________________________________________<br>
SlackBuilds-users mailing list<br>
<a href="mailto:SlackBuilds-users@slackbuilds.org">SlackBuilds-users@slackbuilds.org</a><br>
<a href="http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users" target="_blank">http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users</a><br>
Archives - <a href="http://lists.slackbuilds.org/pipermail/slackbuilds-users/" target="_blank">http://lists.slackbuilds.org/pipermail/slackbuilds-users/</a><br>
FAQ - <a href="http://slackbuilds.org/faq/" target="_blank">http://slackbuilds.org/faq/</a><br>
<br>
</blockquote></div><br></div>