<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif">I've looked over the diff but not your previous script yet.  I agree with the diff so far but I do wonder about why those two blocks of variables are separated.  They really look like they should be together since the logic for determining $ARCH doesn't affect the second block of variables.  Shouldn't the templates be changed to group them together or is there another reason for their separation.  It's a minor and purely aesthetic point, but it just didn't look right to me.<br><br>I'll look at your previous script tomorrow and let you know what I think.<br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 14, 2016 at 5:10 AM, Matteo Bernardini <span dir="ltr"><<a href="mailto:matteo.bernardini@gmail.com" target="_blank">matteo.bernardini@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">2016-03-14 7:31 GMT+01:00 Eric Pratt <<a href="mailto:eric.b.pratt@gmail.com">eric.b.pratt@gmail.com</a>>:<br>
> I see.  Thanks!<br>
><br>
> On Sun, Mar 13, 2016 at 11:02 PM, Willy Sudiarto Raharjo<br>
> <<a href="mailto:willysr@slackbuilds.org">willysr@slackbuilds.org</a>> wrote:<br>
>><br>
>> > Well, it looks like the same guy did it again in January.  I thought<br>
>> > only<br>
>> > the maintainer could submit updates to the slackbuild.  Is that<br>
>> > incorrect?<br>
>> ><br>
>> > Since this appears to be happening, lets just go ahead and commit this<br>
>> > update.  It eliminates the cumbersome patch files and gets us all the<br>
>> > latest bug fixes except for what's in the last release since that<br>
>> > release<br>
>> > wont build and I haven't investigated why yet.<br>
>><br>
>> It was one of the admin, Matteo (ponce)<br>
>> During development cycle like this, we often update the SlackBuild if<br>
>> it's really needed for several reasons:<br>
>> - Fix a build failure due to changes in Slackware<br>
>> - Not yet updated by maintainer for a long time<br>
<br>
</span>Hi Eric,<br>
<br>
as I am using vice myself I noticed when the build broke in the last<br>
5+ years (compiler, ffmpeg versions, etc.) and took care of fixing it<br>
or we had to remove it from the repository.<br>
as for the last update it slipped from my personal repository when<br>
merging the other fixes for current...<br>
<br>
I just had a look at your submission and I see you have rebased your<br>
modifications on your old script and not what we have in our<br>
repository, so I had to fix some things again:<br>
- the homepage isn't working anymore, so I switched to the sourceforge one;<br>
- I moved the block that defines some variables in the slackbuild<br>
lower in it because that's how we have in most of our SlackBuilds:<br>
this helps in case we had to do some automated scripting cleanups in<br>
our repository, like it often happens (this is effective if we have<br>
scripts conforming to the templates);<br>
- I added man and info file compression blocks;<br>
- as the doc directory from the sources folder is messy and the "make<br>
install" already installs a clean one in<br>
/usr/lib$LIBDIRSUFFIX/$PRGNAM/doc where it shouldn't be, I opted to<br>
move this later in /usr/doc/$PRGNAM-$VERSION and the html stuff in a<br>
subdirectory, fixing permissions (executable text files).<br>
I attach the diff over your last submission: if it's ok for you I'll<br>
commit your submission with these corrections in our git.<br>
<span class="HOEnZb"><font color="#888888"><br>
Matteo<br>
</font></span><br>
P.S. Note that not enabling the bundled ffmpeg implies that recording<br>
videos of the emulator running from the menus doesn't work: feel free<br>
to look at the script I previously submitted.<br>
<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" rel="noreferrer" target="_blank">http://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users</a><br>
Archives - <a href="http://lists.slackbuilds.org/pipermail/slackbuilds-users/" rel="noreferrer" target="_blank">http://lists.slackbuilds.org/pipermail/slackbuilds-users/</a><br>
FAQ - <a href="http://slackbuilds.org/faq/" rel="noreferrer" target="_blank">http://slackbuilds.org/faq/</a><br>
<br>
<br></blockquote></div><br></div>