tech-pkg archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: grafana hits pkgsrc limits again



* On 2024-07-10 at 12:19 BST, Thomas Klausner wrote:

I tried that, but there seems to be a problem with the exit status in
a simple test case. With the attached diff:

Not sure, it seems to work fine for me?

-	if cd ${DISTDIR} && ${_CHECKSUM_CMD} ${DISTINFO_FILE:tA} ${_CKSUMFILES}; then \
+	if cd ${DISTDIR}; {						\
+		${_CKSUMFILES:@f@echo ${f};@}				\

It worked fine for me unmodified, but I also then changed this to "cd .. &&" and ${ECHO} instead of echo.

It might also be worth being explicit about the stdin option and adding "-I" support like distinfo.awk has. I've taken advantage of that option with this commit to clean up the filelist handling:

  https://github.com/TritonDataCenter/pkgsrc/commit/59ec2dc8d391b39ee764e3bd3f96d8ef98ca3768

Currently looking at distinfo.awk to see if we can batch process the checksums.

--
Jonathan Perkin   -   mnx.io   -   pkgsrc.smartos.org
Open Source Complete Cloud   www.tritondatacenter.com


Home | Main Index | Thread Index | Old Index