From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 111230 invoked by alias); 29 Apr 2017 12:36:46 -0000 Mailing-List: contact cygwin-apps-help@cygwin.com; run by ezmlm Precedence: bulk Sender: cygwin-apps-owner@cygwin.com List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Mail-Followup-To: cygwin-apps@cygwin.com Received: (qmail 111214 invoked by uid 89); 29 Apr 2017 12:36:45 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.9 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW autolearn=ham version=3.3.2 spammy=fallen, 0.9, DESCRIPTION, schrieb X-HELO: out4-smtp.messagingengine.com Received: from out4-smtp.messagingengine.com (HELO out4-smtp.messagingengine.com) (66.111.4.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sat, 29 Apr 2017 12:36:43 +0000 Received: from compute6.internal (compute6.nyi.internal [10.202.2.46]) by mailout.nyi.internal (Postfix) with ESMTP id 41B8D20ABF; Sat, 29 Apr 2017 08:36:44 -0400 (EDT) Received: from frontend2 ([10.202.2.161]) by compute6.internal (MEProxy); Sat, 29 Apr 2017 08:36:44 -0400 X-ME-Sender: Received: from [192.168.1.102] (host31-51-207-210.range31-51.btcentralplus.com [31.51.207.210]) by mail.messagingengine.com (Postfix) with ESMTPA id DA2832471A; Sat, 29 Apr 2017 08:36:43 -0400 (EDT) Subject: Re: [ITP] man-italic To: Thomas Wolff References: <57EC1DFA.8030103@towo.net> <31684ee3-b4b7-abea-4e82-8f63e4cd106a@towo.net> Cc: cygwin-apps@cygwin.com From: Jon Turney Message-ID: <3ddde0b4-f603-0dec-17c4-5aa1ed7a8e7e@dronecode.org.uk> Date: Sat, 29 Apr 2017 12:36:00 -0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <31684ee3-b4b7-abea-4e82-8f63e4cd106a@towo.net> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2017-04/txt/msg00116.txt.bz2 On 21/04/2017 08:59, Thomas Wolff wrote: > Am 19.04.2017 um 20:38 schrieb Jon Turney: >> On 28/09/2016 20:46, Thomas Wolff wrote: >>> The man-italic wrapper scripts enable italic display in manual pages >>> (where italic is specified in the page) in terminals supporting >>> italic mode. >>> >>> wget http://towo.net/cygwin/man-italic/man-italic-0.9-0.tar.bz2 >>> wget http://towo.net/cygwin/man-italic/man-italic-0.9-src.tar.bz2 >>> wget http://towo.net/cygwin/man-italic/setup.hint >> >> This seems to have fallen through the cracks and been completely >> forgotten. Sorry about that. > Fine you noticed anyway:) > >> Looking at this, I see that this is not a cygport package. While this >> is something which has been done historically, and I know it may seen >> overkill for something this small, it's something I'm very reluctant >> to see in new packages. > As you say, overkill; I've fiddled around with cygport a few times > already and I can't say I'm happy with it. While it seems to be > powerful, especially basic use isn't documented in a basic way... Would > you have a pattern for a script-only package for me, or even ready-to-use? We could certainly do with some commented examples of cygport files in the packaging guide, but SHTDI... Here's an attempt at a man-italic.cygport: NAME=man-italic VERSION=0.9 RELEASE=0 ARCH=noarch LICENSE="Public Domain" CATEGORY="Utils Doc" SUMMARY="Enabling italic display in manual pages" DESCRIPTION="man-italic installs two scripts and hooks them into the workflow of the 'man' command so that the italic attribute of manual pages is actually displayed in italics in terminals that support it." SRC_URI="https://github.com/mintty/man-italic/releases/download/${VERSION}/man-italic-${VERSION}.tar.bz2" SRC_DIR="man-italic" src_compile() { : } src_install() { cd ${S} doman man-italic.7 insinto /usr/share/${NAME} doins grotty iroff insinto /etc/postinstall newins postinstall ${NAME}.sh insinto /etc/preremove newins preremove ${NAME}.sh } >> The license which applies to the original work here needs to be stated. > Checking the package contributor's guide, a license does not seem to be > strictly necessary, so I thought for a simple thing it could go as > public domain implicitly. But if you prefer, I'll attach a gnu to it. Public domain is fine. But we need a statement inside the source package what license applies. I hope you can see that this is a simple consequence of the need to know that we can distribute the package. >> It would be helpful if the description clarified that what this does >> is "show italic text in man pages properly as italic, rather than as >> underlined" > You mean the sdesc one-liner, not ldesc? I'd like to include the > "enabling" aspect because the package does not do the actual display > itself and does not need to be invoked as a tool, like: > sdesc: "Enabling proper italic display of italic text in man pages, > rather than underlined" It's more that I was so used seeing underlined text in my manpages I didn't realize that it was supposed to be italic :-) >> That said, this works and is pretty cool. Nice! > Thanks. Would that be a "GTG" after sorting out the issues, or are still > 3 supporters needed as there used to be? Thinking about this a bit more, I have concerns about the preremove/postinstall scripts. These certainly won't correctly handle the groff package being upgraded. Is it possible to change groff to use alternatives so this package can take over grotty? Or patch this into groff and have this package just install an indicator file which turn on that behaviour?