Thanks Audrius! I appreciate suggestions, and I'll try to make better script. Jmol is my first SB script which I shared with the SB community. :)<br><br><div class="gmail_quote">2011/4/5 Audrius Kažukauskas <span dir="ltr"><<a href="mailto:audrius@neutrino.lt">audrius@neutrino.lt</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">On Sun, 2011-04-03 at 12:33:45 +0200, Ivan Pavicevic wrote:<br>
> This script makes tgz package from Jmol's java binary. The script makes jmol<br>
> branch in /opt and then makes symbolic link to /usr/bin. Also, I made<br>
> jmol.desktop, and added jmol.png as icon, so after installation Jmol appears<br>
> in Applications/Education section (I'm using Xfce).<br>
> It works, but if you find anything wrong in the script, please let me know.<br>
<br>
</div>I had a quick peek into your script and it doesn't look like it was<br>
based on any of SBo templates[0].  I suggest to follow them as closely<br>
as possible.  To handle different program and source file names, you<br>
could use SRC_NAME (or something similar) for the latter one.  The<br>
former should be PRGNAM as written in templates.<br>
<br>
[0] <a href="http://slackbuilds.org/templates/" target="_blank">http://slackbuilds.org/templates/</a><br>
<br>
Documentation in Slackware should be placed in /usr/doc, not<br>
/usr/share/doc.  Icon file should be placed in /usr/share/pixmaps, that<br>
way you won't have to provide a full path for it in .desktop file.<br>
<br>
And there are some issues with .desktop file:<br>
<br>
  $ desktop-file-validate jmol.desktop<br>
  jmol.desktop: warning: key "Encoding" in group "Desktop Entry" is deprecated<br>
  jmol.desktop: error: value "Application;Education" for string list key<br>
  "Categories" in group "Desktop Entry" does not have a semicolon (';')<br>
  as trailing character<br>
<br>
I haven't tried to run it, but overall it looks okay, just needs to be<br>
written more closely to SBo guidelines.<br>
<br>
--<br>
<font color="#888888">Audrius Kažukauskas<br>
</font><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>
<br></blockquote></div><br><br clear="all"><br>-- <br>If you don't care where you are, then you ain't lost<br><br>