Created attachment 147078 [details] shar file for the sysutils/colwide port colwide is a Perl script which displays either 80 or specified number of hash (#) characters on the screen. Can be useful to determine terminal width.
Patch looks good, moving to Patch Ready
Does not fetch: you don't need :dist in MASTER_SITES. You should also use EXTRACT_ONLY= # empty to make it explicit. Finally, I don't like that distfile in not versioned: if it's updated, users will have checksum errors. Distfile mirroring would also be a problem. Better wrap it into e.g. colwide-0.01.tar.gz
Created attachment 147469 [details] Updated shar file for the sysutils/colwide port
Created attachment 147470 [details] Poudriere log file for sysutils/colwide
I have updated the shar file for the proposed sysutils/colwide port. The new file is attached, along with the Poudriere build log used to test compilation of the sysutils/colwide port. -Neel
(In reply to neel from comment #5) > I have updated the shar file for the proposed sysutils/colwide port. The new > file is attached, along with the Poudriere build log used to test > compilation of the sysutils/colwide port. > -Neel I forgot to mention I did not attach the files to comment #5, but to #3 and #4. -Neel
Is there any status on committing the sysutils/colwide port?
===> License BSD2CLAUSE accepted by the user ===> colwide-0.01 depends on file: /usr/local/sbin/pkg - found ===> Fetching all distfiles required by colwide-0.01 for building ===> Extracting for colwide-0.01 => SHA256 Checksum OK for colwide-0.01. ===> Patching for colwide-0.01 ===> Configuring for colwide-0.01 ===> Staging for colwide-0.01 ===> colwide-0.01 depends on file: /usr/local/bin/perl5.18.2 - found ===> Generating temporary packing list install -m 555 /usr/distfiles/colwide-0.01 /tmp/colwide/work/stage/tmp/colwide/prefix/bin/colwide ====> Compressing man pages (compress-man) ====> Running Q/A tests (stage-qa) Error: '/usr/bin/perl' is an invalid shebang you need USES=shebangfix for 'bin/colwide' *** Error code 1 You probably need to copy distfile into ${WRKSRC} on do-extract and add USES=shebangfix to fix perl path in there. Otherwise looks good.
Created attachment 147598 [details] Fixed shar file for the sysutils/colwide port Hi amdmi3@, Related to what you said on Comment #8, I have addressed this shebang error and have uploaded an updated shar file for the sysutils/colwide port. -Neel
There doesn't seem to be any difference between this and previous version of the shar.
Created attachment 147603 [details] Updated and CORRECT shar file for the sysutils/colwide port Hi @amdmi3, Related to what you said on Comment #10, it turned out I did upload the old shar file again. My mistake. I did I upload the correct shar file now. -Neel
Ok, looks good. I forgot one more thing - sysutils doesn't really seem as a suitable category for this, I'd think of misc instead. What do you think?
(In reply to Dmitry Marakasov from comment #12) > Ok, looks good. I forgot one more thing - sysutils doesn't really seem as a > suitable category for this, I'd think of misc instead. What do you think? You have my approval for doing this! If I was doing this again, I would have chose misc instead of sysutils.
A commit references this bug: Author: amdmi3 Date: Wed Sep 24 13:56:27 UTC 2014 New revision: 369180 URL: http://svnweb.freebsd.org/changeset/ports/369180 Log: colwide is a Perl script which displays either 80 or specified number of hash (#) characters on the screen. Can be useful to determine terminal width. WWW: http://www.neelc.org/software/colwide/ PR: 193466 Submitted by: neel@neelc.org Changes: head/misc/Makefile head/misc/colwide/ head/misc/colwide/Makefile head/misc/colwide/distinfo head/misc/colwide/pkg-descr