logo       

Re: MakeMaker movement, new snapshot: msg#00009

Subject: Re: MakeMaker movement, new snapshot
On 2004-04-04, at 16:25:54 -0400, Michael G Schwern wrote:

> On Sun, Apr 04, 2004 at 01:07:17PM +0200, Marcus Holland-Moritz wrote:
> > I guess my "License" patch is part of those. :(
> 
> I think I missed that one and I can't find that in the archives.  

http://www.nntp.perl.org/group/perl.makemaker/1628

> Could you resend?

Sure.

! > On Sun, Dec 14, 2003 at 01:17:49AM +0100, Marcus Holland-Moritz wrote:
! > > Here's a first shot. The attached patch (against the latest repository
! > > snapshot) adds a 'LICENSE' parameter to MakeMaker. It also tries to make
! > > the metafile output a little more Module::Build-ish. I didn't add a
! > > parameter for 'DISTRIBUTION_TYPE', since this is currently hardcoded
! > > also in M::B.
! > > 
! > > As this is my first patch to MM, please let me know if I missed anything.
! > 
! > Looks good at first glance.  My only comment would be that it would
! > be nice if the license warning could happen only as the META.yml is
! > generated.  This will make MM forward compatible should we add more
! > licenses, so if an older MM installs a module generated with a new license
! > type it won't warn.
! 
! Sounds reasonable.
! 
! > That'll probably require moving the license check code into
! > an ExtUtils::MM::Command function so the check can be done in the
! > metafile target.
! 
! Updated patch attached. I also patched the C<oneliner> subs to add
! a '--' at the end of the options list. This makes it possible for
! oneliners to be passed arguments that start with dashes, which is
! required e.g. on Win32 when doing:
! 
!     ECHO = $(PERLRUN) -l -e "print qq{@ARGV}" --
!     [...]
!     metafile :
!             $(NOECHO) $(ECHO) "--- #YAML:1.0" > META_new.yml
! 
! This wouldn't work without the '--'.
! 
! -- Marcus

> -- 
> Michael G Schwern        schwern@xxxxxxxxx  http://www.pobox.com/~schwern/
> michael schwern is
> 


-- 
You do not have mail.

Attachment: mm_meta_license-2.diff
Description: Text Data

<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

Recently Viewed:
science.linguis...    culture.sf.lite...    video.mplayer.c...    yellowdog.gener...    ietf.rfc822/199...    emacs.help/2002...    redhat.release....    kernel.speakup/...    java.openejb.de...    debian.devel.gt...    xfree86.newbie/...    bug-tracking.ma...    pam/2003-05/msg...    games.devel.ope...    user-groups.lin...    music.pancham/2...    network.mq.deve...    web.html.genera...    arklinux.bugs/2...    linux.ecasound/...    qnx.openqnx.dev...    org.user-groups...    file-systems.sf...    trustix.contrib...   
Home | blog view | USPTO Patent Archive | advertise | OSDir is an inevitable website. super tiny logo

Free Magazines

Cisco News
Receive a free quarterly e-newsletter with exclusive articles on how Cisco IT uses its own products and solutions to enable the business.
subscribe

Systems Management News, the newspaper for IT systems administration and data center managers! Each issue of Systems Management News is chock-full of news and analysis to help you understand what's happening in your field.
subscribe

The Enterprise Newsweekly eWeek is the essential technology information source for builders of e-business.
subscribe

Oracle Magazine Oracle Magazine contains technology strategy articles, sample code, tips, Oracle and partner news, how to articles for developers and DBAs, and more. Oracle (NASDAQ: ORCL) is the world's largest enterprise software company.
subscribe

Total Telecom Total Telecom is "The Economist of the communications industry".
subscribe