You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@bloodhound.apache.org by Ryan Ollos <ry...@wandisco.com> on 2014/02/21 07:18:27 UTC

Re: svn commit: r1570462 - in /bloodhound/trunk: bloodhound_dashboard/setup.cfg bloodhound_multiproduct/setup.cfg bloodhound_relations/setup.cfg bloodhound_search/setup.cfg bloodhound_theme/setup.cfg

On Thu, Feb 20, 2014 at 10:14 PM, <rj...@apache.org> wrote:

> Author: rjollos
> Date: Fri Feb 21 06:14:36 2014
> New Revision: 1570462
>
> URL: http://svn.apache.org/r1570462
> Log:
> 0.8dev: Set copyright holder for the catalogs.
>
> Modified:
>     bloodhound/trunk/bloodhound_dashboard/setup.cfg
>     bloodhound/trunk/bloodhound_multiproduct/setup.cfg
>     bloodhound/trunk/bloodhound_relations/setup.cfg
>     bloodhound/trunk/bloodhound_search/setup.cfg
>     bloodhound/trunk/bloodhound_theme/setup.cfg
>
> Modified: bloodhound/trunk/bloodhound_dashboard/setup.cfg
> URL:
> http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_dashboard/setup.cfg?rev=1570462&r1=1570461&r2=1570462&view=diff
>
> ==============================================================================
> --- bloodhound/trunk/bloodhound_dashboard/setup.cfg (original)
> +++ bloodhound/trunk/bloodhound_dashboard/setup.cfg Fri Feb 21 06:14:36
> 2014
> @@ -25,7 +25,8 @@ formats = gztar,bztar,ztar,tar,zip
>
>  [extract_messages]
>  add_comments = TRANSLATOR:
> -msgid_bugs_address =
> +copyright_holder = Apache Software Foundation
> +msgid_bugs_address = dev@bloodhound.apache.org
>  output_file = bhdashboard/locale/messages.pot
>  keywords = _ ngettext:1,2 N_ tag_ tagn_:1,2 Option:4 BoolOption:4
> IntOption:4 ListOption:6 ExtensionOption:5 PathOption:4
>  no-wrap = true
>
> Modified: bloodhound/trunk/bloodhound_multiproduct/setup.cfg
> URL:
> http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_multiproduct/setup.cfg?rev=1570462&r1=1570461&r2=1570462&view=diff
>
> ==============================================================================
> --- bloodhound/trunk/bloodhound_multiproduct/setup.cfg (original)
> +++ bloodhound/trunk/bloodhound_multiproduct/setup.cfg Fri Feb 21 06:14:36
> 2014
> @@ -25,7 +25,8 @@ formats = gztar,bztar,ztar,tar,zip
>
>  [extract_messages]
>  add_comments = TRANSLATOR:
> -msgid_bugs_address =
> +copyright_holder = Apache Software Foundation
> +msgid_bugs_address = dev@bloodhound.apache.org
>  output_file = multiproduct/locale/messages.pot
>  keywords = _ ngettext:1,2 N_ tag_ tagn_:1,2 Option:4 BoolOption:4
> IntOption:4 ListOption:6 ExtensionOption:5 PathOption:4
>  no-wrap = true
>
> Modified: bloodhound/trunk/bloodhound_relations/setup.cfg
> URL:
> http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_relations/setup.cfg?rev=1570462&r1=1570461&r2=1570462&view=diff
>
> ==============================================================================
> --- bloodhound/trunk/bloodhound_relations/setup.cfg (original)
> +++ bloodhound/trunk/bloodhound_relations/setup.cfg Fri Feb 21 06:14:36
> 2014
> @@ -25,7 +25,8 @@ formats = gztar,bztar,ztar,tar,zip
>
>  [extract_messages]
>  add_comments = TRANSLATOR:
> -msgid_bugs_address =
> +copyright_holder = Apache Software Foundation
> +msgid_bugs_address = dev@bloodhound.apache.org
>  output_file = bhrelations/locale/messages.pot
>  keywords = _ ngettext:1,2 N_ tag_ tagn_:1,2 Option:4 BoolOption:4
> IntOption:4 ListOption:6 OrderedExtensionsOption:6 PathOption:4
>  no-wrap = true
>
> Modified: bloodhound/trunk/bloodhound_search/setup.cfg
> URL:
> http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_search/setup.cfg?rev=1570462&r1=1570461&r2=1570462&view=diff
>
> ==============================================================================
> --- bloodhound/trunk/bloodhound_search/setup.cfg (original)
> +++ bloodhound/trunk/bloodhound_search/setup.cfg Fri Feb 21 06:14:36 2014
> @@ -25,7 +25,8 @@ formats = gztar,bztar,ztar,tar,zip
>
>  [extract_messages]
>  add_comments = TRANSLATOR:
> -msgid_bugs_address =
> +copyright_holder = Apache Software Foundation
> +msgid_bugs_address = dev@bloodhound.apache.org
>  output_file = bhsearch/locale/messages.pot
>  keywords = _ ngettext:1,2 N_ tag_ tagn_:1,2 Option:4 BoolOption:4
> IntOption:4 ListOption:6 ExtensionOption:5 OrderedExtensionsOption:6
> PathOption:4
>  no-wrap = true
>
> Modified: bloodhound/trunk/bloodhound_theme/setup.cfg
> URL:
> http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_theme/setup.cfg?rev=1570462&r1=1570461&r2=1570462&view=diff
>
> ==============================================================================
> --- bloodhound/trunk/bloodhound_theme/setup.cfg (original)
> +++ bloodhound/trunk/bloodhound_theme/setup.cfg Fri Feb 21 06:14:36 2014
> @@ -25,7 +25,8 @@ formats = gztar,bztar,ztar,tar,zip
>
>  [extract_messages]
>  add_comments = TRANSLATOR:
> -msgid_bugs_address =
> +copyright_holder = Apache Software Foundation
> +msgid_bugs_address = dev@bloodhound.apache.org
>  output_file = bhtheme/locale/messages.pot
>  keywords = _ ngettext:1,2 N_ tag_ tagn_:1,2 Option:4 BoolOption:4
> IntOption:4 ListOption:6 ExtensionOption:5 PathOption:4
>  no-wrap = true
> @@ -46,8 +47,8 @@ domain = bhtheme
>
>  [extract_messages_js]
>  add_comments = TRANSLATOR:
> -copyright_holder =
> -msgid_bugs_address =
> +copyright_holder = Apache Software Foundation
> +msgid_bugs_address = dev@bloodhound.apache.org
>  output_file = bhtheme/locale/messages-js.pot
>  keywords = _ ngettext:1,2 N_
>  mapping_file = messages-js.cfg
>
>
>
After r1570462, the pot file header is now:

# Translations template for BloodhoundDashboardPlugin.
# Copyright (C) 2014 Apache Software Foundation
# This file is distributed under the same license as the
BloodhoundDashboardPlugin project.

rather than,

# Translations template for BloodhoundDashboardPlugin.
# Copyright (C) 2014 ORGANIZATION
# This file is distributed under the same license as the
BloodhoundDashboardPlugin project.

My aim is to address the deficiencies in the RAT report:
http://ci.apache.org/projects/bloodhound/rat-output.html

I am not sure if the header is enough per the Apache recommendations (will
investigate).

If we need to set the header comment with the typical license header test,
that may be difficult. The `header_comment` can be specified when
instantiating the catalog (1), but not through the `extract_messages`
helper that we utilize (2).

(1)
http://babel.pocoo.org/docs/api/messages/catalog/?highlight=header_comment
(2) http://babel.pocoo.org/docs/setup/?highlight=extract_messages

Re: svn commit: r1570462 - in /bloodhound/trunk: bloodhound_dashboard/setup.cfg bloodhound_multiproduct/setup.cfg bloodhound_relations/setup.cfg bloodhound_search/setup.cfg bloodhound_theme/setup.cfg

Posted by Ryan Ollos <ry...@wandisco.com>.
On Fri, Feb 21, 2014 at 8:56 AM, Olemis Lang <ol...@gmail.com> wrote:

> [...]
>
> > >
> > >
> > >
> > After r1570462, the pot file header is now:
> >
> > # Translations template for BloodhoundDashboardPlugin.
> > # Copyright (C) 2014 Apache Software Foundation
> > # This file is distributed under the same license as the
> > BloodhoundDashboardPlugin project.
> >
> > rather than,
> >
> > # Translations template for BloodhoundDashboardPlugin.
> > # Copyright (C) 2014 ORGANIZATION
> > # This file is distributed under the same license as the
> > BloodhoundDashboardPlugin project.
> >
>
> \o/
>
> > My aim is to address the deficiencies in the RAT report:
> > http://ci.apache.org/projects/bloodhound/rat-output.html
> >
>
> +1
>
> > I am not sure if the header is enough per the Apache recommendations
> (will
> > investigate).
> >
> > If we need to set the header comment with the typical license header
> test,
> > that may be difficult. The `header_comment` can be specified when
> > instantiating the catalog (1), but not through the `extract_messages`
> > helper that we utilize (2).
> >
> > (1)
> >
> http://babel.pocoo.org/docs/api/messages/catalog/?highlight=header_comment
> > (2) http://babel.pocoo.org/docs/setup/?highlight=extract_messages
>
> ... but I guess something can be done if we contact babel devs for support.
> In advance it seems to me that ALv2 license header (i.e. comments) will be
> required. If not then we could just ignore these, but I'm keen to insert
> them.
>

That is a good idea. I've opened a ticket:
https://github.com/mitsuhiko/babel/issues/82

Re: svn commit: r1570462 - in /bloodhound/trunk: bloodhound_dashboard/setup.cfg bloodhound_multiproduct/setup.cfg bloodhound_relations/setup.cfg bloodhound_search/setup.cfg bloodhound_theme/setup.cfg

Posted by Olemis Lang <ol...@gmail.com>.
[...]

> >
> >
> >
> After r1570462, the pot file header is now:
>
> # Translations template for BloodhoundDashboardPlugin.
> # Copyright (C) 2014 Apache Software Foundation
> # This file is distributed under the same license as the
> BloodhoundDashboardPlugin project.
>
> rather than,
>
> # Translations template for BloodhoundDashboardPlugin.
> # Copyright (C) 2014 ORGANIZATION
> # This file is distributed under the same license as the
> BloodhoundDashboardPlugin project.
>

\o/

> My aim is to address the deficiencies in the RAT report:
> http://ci.apache.org/projects/bloodhound/rat-output.html
>

+1

> I am not sure if the header is enough per the Apache recommendations (will
> investigate).
>
> If we need to set the header comment with the typical license header test,
> that may be difficult. The `header_comment` can be specified when
> instantiating the catalog (1), but not through the `extract_messages`
> helper that we utilize (2).
>
> (1)
> http://babel.pocoo.org/docs/api/messages/catalog/?highlight=header_comment
> (2) http://babel.pocoo.org/docs/setup/?highlight=extract_messages

... but I guess something can be done if we contact babel devs for support.
In advance it seems to me that ALv2 license header (i.e. comments) will be
required. If not then we could just ignore these, but I'm keen to insert
them.

Sent from Android

--
Regards

Olemis - @olemislc
Blog-ES : http://simelo-es.blogspot.com
Blog-EN : http://simelo-en.blogspot.com
Projects : http://blood-hound.net