Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 21 May 1995 18:42:50 -0700
From:      asami@cs.berkeley.edu (Satoshi Asami | =?ISO-2022-JP?B?GyRCQHUbKEI=?= =?ISO-2022-JP?B?GyRCOCsbKEIgGyRCOC0bKEI=?=)
To:        jhs@freefall.cdrom.com
Cc:        CVS-commiters@freefall.cdrom.com, cvs-ports@freefall.cdrom.com
Subject:   Re: cvs commit: ports/comms/hylafax - Imported sources
Message-ID:  <199505220142.SAA18778@silvia.HIP.Berkeley.EDU>
In-Reply-To: <199505212237.PAA00464@freefall.cdrom.com> (message from Julian Stacey on Sun, 21 May 1995 15:37:50 -0700)

next in thread | previous in thread | raw e-mail | index | archive | help
 *   Reviewed by:	Earlier draft reviewed by Satoshi (Thanks)

Julian, I asked you to send me a mail when you are done, not to commit
it.  All I did at that time was to make some comments on your 100+
line Makefile, I haven't taken a look at any of your numerous
auxiliary scripts and I haven't done any testing.

I regret your decision to ignore what I said.

 *   I will maintain ports/comms/hylafax as I need fax capability.

You haven't demonstrated you can maintain the port (see below), this
isn't an excuse to import something I can't understand.

 *   Thanks to Asami Satoshi for suggested improvements,
 *   I leave him to `bless' this port by adding the final
 *           comms/Makefile: SUBDIR += hylafax
 *   Until he does that, any mistakes I may have made shouldnt break
 * any makes :-) 

You don't understand.  I do NOT want a broken or messy port in the
tree.  Whether it's referred to by the parent Makefile is irrelevant,
and I want NO ports in the BROKEN list, that list exists for things
that worked before but don't anymore because of one reason or another,
not to squeeze in things that are known to be broken to begin with.

I tried to fix the Makefile now, added EXTRACT_SUFX and deleted the
useless echo statements in the pre-fetch target, re-made the checksums
but it now has 17 (!) failed hunks of patches.

If you can't fix this within 24 hours, expect it to be purged from the 
repository and replaced by another, simpler and working version.

Satoshi



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199505220142.SAA18778>