You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@perl.apache.org by "Philippe M. Chiasson" <go...@ectoplasm.org> on 2004/06/29 00:00:21 UTC

[mp2 Patch] META.yml

As far as I know, the CPAN indexer now supports the no_index META.yml
field proprely, so I think maintaining our own META.yml from now on
is in order.

Index: .cvsignore
===================================================================
RCS file: /home/cvs/modperl-2.0/.cvsignore,v
retrieving revision 1.10
diff -u -I$Id -r1.10 .cvsignore
--- .cvsignore	23 Dec 2003 04:26:08 -0000	1.10
+++ .cvsignore	28 Jun 2004 21:59:22 -0000
@@ -9,5 +9,4 @@
  MANIFEST
  glue_pods
  .mypacklist
-META.yml

Index: Changes
===================================================================
RCS file: /home/cvs/modperl-2.0/Changes,v
retrieving revision 1.392
diff -u -I$Id -r1.392 Changes
--- Changes	27 Jun 2004 21:26:45 -0000	1.392
+++ Changes	28 Jun 2004 21:59:22 -0000
@@ -12,6 +12,9 @@

  =item 1.99_15-dev

+META.yml is now included and marks Apache-Test as a private resource
+for CPAN and friends [Gozer]
+
  Registry: remove the misleading prefix "$$: $class:" in the logged
  error message, since not only registry errors will get logged if $@ is
  set [Stas]
Index: META.yml
===================================================================
RCS file: META.yml
diff -N META.yml
--- /dev/null	1 Jan 1970 00:00:00 -0000
+++ META.yml	28 Jun 2004 21:59:22 -0000
@@ -0,0 +1,12 @@
+--- #YAML:1.0
+# http://module-build.sourceforge.net/META-spec.html
+#XXXXXXX This is a prototype!!!  It will change in the future!!! XXXXX#
+name:              mod_perl
+version_from:      lib/mod_perl.pm
+installdirs:       site
+license:           asf
+distribution_type: module
+
+no_index:
+    directory:
+        - Apache-Test
Index: Makefile.PL
===================================================================
RCS file: /home/cvs/modperl-2.0/Makefile.PL,v
retrieving revision 1.140
diff -u -I$Id -r1.140 Makefile.PL
--- Makefile.PL	27 Jun 2004 23:41:06 -0000	1.140
+++ Makefile.PL	28 Jun 2004 21:59:22 -0000
@@ -68,6 +68,7 @@
      ABSTRACT_FROM => 'lib/mod_perl.pm',
      EXE_FILES     => \@exe_files,
      DEFINE        => get_DEFINE(),
+    NO_META       => 1,
      macro         => {
          MODPERL_SRC        => $code->path,
          MODPERL_MAKEFILE   => basename($build->default_file('makefile')),

-- 
--------------------------------------------------------------------------------
Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5
http://gozer.ectoplasm.org/     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

Re: [mp2 Patch] META.yml

Posted by Stas Bekman <st...@stason.org>.
Philippe M. Chiasson wrote:
> Stas Bekman wrote:
> 
>> Philippe M. Chiasson wrote:
>> [...]
>>
>>>> but I can't see no_index being documented at the above URL. I 
>>>> remember we discussed a different name for this attribute at p5p 
>>>> (i.e. 'private').
>>>
>>>
>>>
>>>
>>> It has not been made part of the 'official' META.yml spec, but it's 
>>> apparently
>>> already supported by CPAN and used by more than one module
>>>
>>> http://www.mail-archive.com/cpan-discuss@perl.org/msg00054.html
>>>
>>> Does seem to indicate 'private' would be a better choice though.
>>
>>
>>
>> May be someone should ask again what's the status of things. I haven't 
>> been pushing for those changes for quite some time now.
> 
> 
> Well, I've just noticed that perl-current does have a top level META.yml 
> now, and it does make use of the 'private' key. So, I guess
> it might be possible to assume that if perl does it, it's all good ;-)

It could be not the case, since PAUSE treats perl core differently.

>> But go ahead and commit your original patch, so it's not going to be 
>> wasted and we can adjust it later.
> 
> It's been taken care in it's current form.

Great!


-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org


Re: [mp2 Patch] META.yml

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Stas Bekman wrote:

> Philippe M. Chiasson wrote:
> [...]
>
>>> but I can't see no_index being documented at the above URL. I 
>>> remember we discussed a different name for this attribute at p5p 
>>> (i.e. 'private').
>>
>>
>>
>> It has not been made part of the 'official' META.yml spec, but it's 
>> apparently
>> already supported by CPAN and used by more than one module
>>
>> http://www.mail-archive.com/cpan-discuss@perl.org/msg00054.html
>>
>> Does seem to indicate 'private' would be a better choice though.
>
>
> May be someone should ask again what's the status of things. I haven't 
> been pushing for those changes for quite some time now.

Well, I've just noticed that perl-current does have a top level META.yml 
now, and it does make use of the 'private' key. So, I guess
it might be possible to assume that if perl does it, it's all good ;-)

> But go ahead and commit your original patch, so it's not going to be 
> wasted and we can adjust it later.

It's been taken care in it's current form.


Re: [mp2 Patch] META.yml

Posted by Stas Bekman <st...@stason.org>.
Philippe M. Chiasson wrote:
[...]
>> but I can't see no_index being documented at the above URL. I remember 
>> we discussed a different name for this attribute at p5p (i.e. 'private').
> 
> 
> It has not been made part of the 'official' META.yml spec, but it's 
> apparently
> already supported by CPAN and used by more than one module
> 
> http://www.mail-archive.com/cpan-discuss@perl.org/msg00054.html
> 
> Does seem to indicate 'private' would be a better choice though.

May be someone should ask again what's the status of things. I haven't been 
pushing for those changes for quite some time now.

But go ahead and commit your original patch, so it's not going to be wasted 
and we can adjust it later.


-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org


Re: [mp2 Patch] META.yml

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.

Stas Bekman wrote:
> Philippe M. Chiasson wrote:
> 
>>As far as I know, the CPAN indexer now supports the no_index META.yml
>>field proprely, so I think maintaining our own META.yml from now on
>>is in order.
>>
>>Index: .cvsignore
>>===================================================================
>>RCS file: /home/cvs/modperl-2.0/.cvsignore,v
>>retrieving revision 1.10
>>diff -u -I$Id -r1.10 .cvsignore
>>--- .cvsignore    23 Dec 2003 04:26:08 -0000    1.10
>>+++ .cvsignore    28 Jun 2004 21:59:22 -0000
>>@@ -9,5 +9,4 @@
>> MANIFEST
>> glue_pods
>> .mypacklist
>>-META.yml
>>
>>Index: Changes
>>===================================================================
>>RCS file: /home/cvs/modperl-2.0/Changes,v
>>retrieving revision 1.392
>>diff -u -I$Id -r1.392 Changes
>>--- Changes    27 Jun 2004 21:26:45 -0000    1.392
>>+++ Changes    28 Jun 2004 21:59:22 -0000
>>@@ -12,6 +12,9 @@
>>
>> =item 1.99_15-dev
>>
>>+META.yml is now included and marks Apache-Test as a private resource
>>+for CPAN and friends [Gozer]
>>+
>> Registry: remove the misleading prefix "$$: $class:" in the logged
>> error message, since not only registry errors will get logged if $@ is
>> set [Stas]
>>Index: META.yml
>>===================================================================
>>RCS file: META.yml
>>diff -N META.yml
>>--- /dev/null    1 Jan 1970 00:00:00 -0000
>>+++ META.yml    28 Jun 2004 21:59:22 -0000
>>@@ -0,0 +1,12 @@
>>+--- #YAML:1.0
>>+# http://module-build.sourceforge.net/META-spec.html
>>+#XXXXXXX This is a prototype!!!  It will change in the future!!! XXXXX#
>>+name:              mod_perl
>>+version_from:      lib/mod_perl.pm
>>+installdirs:       site
>>+license:           asf
>>+distribution_type: module
>>+
>>+no_index:
>>+    directory:
>>+        - Apache-Test
> 
> 
> +1
> 
> but I can't see no_index being documented at the above URL. I remember we 
> discussed a different name for this attribute at p5p (i.e. 'private').

It has not been made part of the 'official' META.yml spec, but it's apparently
already supported by CPAN and used by more than one module

http://www.mail-archive.com/cpan-discuss@perl.org/msg00054.html

Does seem to indicate 'private' would be a better choice though.

> there are a few other useful keys: requires, recommends and build_requires. 
> The latter two are fine. But the first one will require a duplicate, which 
> sucks as it's easy to get the two things out of sync.

Yes, I am aware of this and I'd suggest we keep that stuff in the Makefile.PL
the way it is right now and not use META.yml for that purpose. At least not
yet.

-- 
--------------------------------------------------------------------------------
Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5
http://gozer.ectoplasm.org/     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

Re: [mp2 Patch] META.yml

Posted by Stas Bekman <st...@stason.org>.
Philippe M. Chiasson wrote:
> As far as I know, the CPAN indexer now supports the no_index META.yml
> field proprely, so I think maintaining our own META.yml from now on
> is in order.
> 
> Index: .cvsignore
> ===================================================================
> RCS file: /home/cvs/modperl-2.0/.cvsignore,v
> retrieving revision 1.10
> diff -u -I$Id -r1.10 .cvsignore
> --- .cvsignore    23 Dec 2003 04:26:08 -0000    1.10
> +++ .cvsignore    28 Jun 2004 21:59:22 -0000
> @@ -9,5 +9,4 @@
>  MANIFEST
>  glue_pods
>  .mypacklist
> -META.yml
> 
> Index: Changes
> ===================================================================
> RCS file: /home/cvs/modperl-2.0/Changes,v
> retrieving revision 1.392
> diff -u -I$Id -r1.392 Changes
> --- Changes    27 Jun 2004 21:26:45 -0000    1.392
> +++ Changes    28 Jun 2004 21:59:22 -0000
> @@ -12,6 +12,9 @@
> 
>  =item 1.99_15-dev
> 
> +META.yml is now included and marks Apache-Test as a private resource
> +for CPAN and friends [Gozer]
> +
>  Registry: remove the misleading prefix "$$: $class:" in the logged
>  error message, since not only registry errors will get logged if $@ is
>  set [Stas]
> Index: META.yml
> ===================================================================
> RCS file: META.yml
> diff -N META.yml
> --- /dev/null    1 Jan 1970 00:00:00 -0000
> +++ META.yml    28 Jun 2004 21:59:22 -0000
> @@ -0,0 +1,12 @@
> +--- #YAML:1.0
> +# http://module-build.sourceforge.net/META-spec.html
> +#XXXXXXX This is a prototype!!!  It will change in the future!!! XXXXX#
> +name:              mod_perl
> +version_from:      lib/mod_perl.pm
> +installdirs:       site
> +license:           asf
> +distribution_type: module
> +
> +no_index:
> +    directory:
> +        - Apache-Test

+1

but I can't see no_index being documented at the above URL. I remember we 
discussed a different name for this attribute at p5p (i.e. 'private').

there are a few other useful keys: requires, recommends and build_requires. 
The latter two are fine. But the first one will require a duplicate, which 
sucks as it's easy to get the two things out of sync.

-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org