<p dir="ltr">Really the only problem is how the SLACKBUILDS.TXT presents the md5sums for those cases, which is inconsistent with the rest of the md5sums in SLACKBUILDS.TXT.</p>
<p dir="ltr">I was trying to leverage SLACKBUILDS.TXT with some parsing scripts and expected to find whitespace there. I could use the info files instead to get the correct md5sum info, I just wanted to raise the issue to the sbo list in case there was something better that could be done.</p>
<p dir="ltr">It's not really the way the .info files are formatted, as you point out with where the whitespace is on the next line. I guess it's more about something wrong with how that SLACKBUILDS.TXT is parsing the info files to generate those entries. Maybe that's a better place to fix it, so additional requirements about md5sum formatting in info files aren't needed.</p>
<br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 6, 2024, 03:13 B. Watson <<a href="mailto:urchlay@slackware.uk">urchlay@slackware.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On Tue, 6 Aug 2024, Bob Funk wrote:<br>
<br>
> Perhaps enforcing spaces before backslash/escape characters in .info files should be done so the files are consistent? Could even be added to sbolint I suppose.<br>
> I'm not sure what is generating that SLACKBUILDS.TXT but perhaps its parsing code could also be adjusted to allow for those cases as an alternate fix.<br>
<br>
sbolint doesn't check for this... it does check that there are spaces<br>
between multiple md5sums, but in the case of vst3sdk, there *are*<br>
spaces, they're just on the next line (indentation on the line after<br>
the \).<br>
<br>
I could add an explicit check, wouldn't be much trouble. Looks like we<br>
have 4 builds in the repo with this problem:<br>
<br>
academic/gephi<br>
development/openjdk8<br>
haskell/haskell-test-framework<br>
libraries/vst3sdk<br>
<br>
Another check that should maybe be added: do we want to *require*<br>
indendation on continuation lines? Example:<br>
<br>
DOWNLOAD="<a href="http://whatever.com/blah.tgz" rel="noreferrer noreferrer" target="_blank">http://whatever.com/blah.tgz</a> \<br>
<a href="http://other.com/foo.tgz" rel="noreferrer noreferrer" target="_blank">http://other.com/foo.tgz</a>"<br>
<br>
The 2nd line ought to be indented. Same thing could happen with<br>
md5sums... right now, these don't get checked for, and really they<br>
don't cause any problems for tools like sbopkg or sboinstall. So how<br>
picky should we be about whitespace?<br>
_______________________________________________<br>
SlackBuilds-users mailing list<br>
<a href="mailto:SlackBuilds-users@slackbuilds.org" target="_blank" rel="noreferrer">SlackBuilds-users@slackbuilds.org</a><br>
<a href="https://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users" rel="noreferrer noreferrer" target="_blank">https://lists.slackbuilds.org/mailman/listinfo/slackbuilds-users</a><br>
Archives - <a href="https://lists.slackbuilds.org/pipermail/slackbuilds-users/" rel="noreferrer noreferrer" target="_blank">https://lists.slackbuilds.org/pipermail/slackbuilds-users/</a><br>
FAQ - <a href="https://slackbuilds.org/faq/" rel="noreferrer noreferrer" target="_blank">https://slackbuilds.org/faq/</a><br>
<br>
</blockquote></div>