Bug 193466
| Summary: | [NEW PORT] sysutils/colwide: A utility which outputs a number of characters to the screen | ||
|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Neel Chauhan <nc> |
| Component: | Individual Port(s) | Assignee: | Dmitry Marakasov <amdmi3> |
| Status: | Closed FIXED | ||
| Severity: | Affects Many People | CC: | amdmi3, mva |
| Priority: | Normal | Keywords: | easy |
| Version: | Latest | ||
| Hardware: | Any | ||
| OS: | Any | ||
| Attachments: | |||
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 |
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.