<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 14, 2017 at 10:31 PM, Daniel Prosser <span dir="ltr"><<a href="mailto:dpross1100@msn.com" target="_blank">dpross1100@msn.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span style="font-size:12.8px">I haven't tried it myself, but this also sounds similar to</span><br style="font-size:12.8px"><span style="font-size:12.8px">sbo-templates:</span><br style="font-size:12.8px"><br style="font-size:12.8px"><a href="https://github.com/dslackw/sbo-templates" rel="noreferrer" target="_blank" style="font-size:12.8px">https://github.com/dslackw/<wbr>sbo-templates</a><br style="font-size:12.8px"><br style="font-size:12.8px"><span style="font-size:12.8px">Which is also on SlackBuilds.org. Not sure if you've seen that.</span></blockquote><div><br></div><div>I actually wasn't aware of that. I didn't think to actually look for other projects providing this. But I think I do prefer a single shell script to handle it rather than a python script (mainly because I don't know python very well. But thanks for the info.</div><div><br></div><div>On Mon, Apr 17, 2017 at 11:46 PM, Daniel Prosser <span dir="ltr"><<a href="mailto:dpross1100@msn.com" target="_blank">dpross1100@msn.com</a>></span> wrote:</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I just gave it a quick try. It seems like a useful script when preparing<br>
a new submission. My test used the cmake template. Here are a few issues<br>
I noticed:<br>
<br>
1. I specified a single requirement using the -r option, but it was<br>
absent from the .info file generated. I tried with and without quotes<br>
around the requirement.<br>
<br>
The next two are some inconsistencies between the generated SlackBuild<br>
script and the template from SlackBuilds.org:<br>
<br>
2. Compress manpages command. The template on SBo has this:<br>
for i in $( find $PKG/usr/man -type l ) ; do ln -s $( readlink $i ).gz $i.gz ; rm $i ; done<br>
<br>
In the generated script:<br>
for i in $( find /usr/man -type l ) ; do ln -s $( readlink  ).gz $i.gz ; rm $i ; done<br>
<br>
>From a quick glance at that line, it looks like your script is not<br>
handling variables inside other variables. Possibly need to use an<br>
escape before a $ somewhere.<br>
<br>
3. The perllocal.pod removal command doesn't have the "|| true" at the<br>
end like the SBo template.</blockquote><div><br></div><div>1. Looks like I used $REQUIRE in one instance and $REQUIRES in the other.</div><div><br></div><div>2. Oops, I missed an escape in there for the $PKG variable. I'm going to go through and see if I missed any others (I did). Hopefully I got them all this time.</div><div><br></div><div>3. This looks to be due to differences in the SBo templates. Both the cmake and perl templates include the "|| true"  at the end of that find, but the autotools doesn't (autotools also has that find span multiple lines instead of the one liner version on the cmake and perl templates. I used the autotools as the base template for that section. I'll add the "|| true" at the end and make it a single line to match the cmake and perl templates.</div><div><br></div><div>autotools:<br><font face="monospace, monospace" style="background-color:rgb(204,204,204)"><span style="color:rgb(0,0,0);white-space:pre-wrap">find $PKG -name perllocal.pod \
</span><span style="color:rgb(0,0,0);white-space:pre-wrap">  -o -name ".packlist" \
</span><span style="color:rgb(0,0,0);white-space:pre-wrap">  -o -name "*.bs" \
</span><span style="color:rgb(0,0,0);white-space:pre-wrap">  | xargs rm -f</span></font><span style="color:rgb(0,0,0);white-space:pre-wrap"><br></span><br></div><div>cmake/perl:<br><span style="color:rgb(0,0,0);white-space:pre-wrap;background-color:rgb(204,204,204)"><font face="monospace, monospace">find $PKG -name perllocal.pod -o -name ".packlist" -o -name "*.bs" | xargs rm -f || true</font></span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap;background-color:rgb(204,204,204)"><font face="monospace, monospace"><br></font></span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap;background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif">Thanks for the report! And for trying it out. These issues should all be fixed now.</font></span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap;background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif"><br></font></span></div><div><span style="color:rgb(0,0,0);white-space:pre-wrap;background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif">Jeremy</font></span></div></div></div></div>