<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif">I was going to try to go to bed, but I decided to hell with it and took a look at the script tonight.  I like your ffmpeg changes so we should probably add those back in.  Also, the diff you sent doesn't seem to include your previous find line for the symbol stripping.<br><br><div class="">-find $PKG | xargs file | grep -e "executable" -e "shared object" | grep ELF \</div><div class="">+find $PKG -print0 | xargs -0 file | grep -e "executable" -e "shared object" | grep ELF \<br><br></div><div class="">I think your version is better and we should probably go with that.  If you'd like, I can put together a updated slackbuild and email it to this thread.  Or if you prefer, you can make the changes in your repository and push them through on your next commit.  We'll go with whatever you think is best.<br></div><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">Thanks for all the help on this!<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif">Eric<br></div><div class="gmail_default" style="font-family:tahoma,sans-serif"><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 15, 2016 at 12:57 AM, Eric Pratt <span dir="ltr"><<a href="mailto:eric.b.pratt@gmail.com" target="_blank">eric.b.pratt@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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"><div><div class="h5">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></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><span>2016-03-14 7:31 GMT+01:00 Eric Pratt <<a href="mailto:eric.b.pratt@gmail.com" target="_blank">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" target="_blank">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><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></div></div><span class="">_______________________________________________<br>
SlackBuilds-users mailing list<br>
<a href="mailto:SlackBuilds-users@slackbuilds.org" target="_blank">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></span></blockquote></div><br></div>
</blockquote></div><br></div>