On Wed, 19 Sep 2018 06:45:13 -0600, Gary Gregory wrote:
The difference is to account for artifact ids that contain a version like commons-lang3. The component id is then just commons-lang. You must not have versions in names for certain names like in the download page. This
change will probably break builds like pool, dbcp, lang, and so on.

Hmm, this completes the confusion!

If "artifactId" is e.g. commons-lang3 then I don't understand how
the reverted line works because (AFAICT) the SVN URL is
https://svn.apache.org/repos/infra/websites/production/commons/content/proper/commons-lang/

The [Math] POM contains these lines:
---CUT---
    <!-- Do not change: "math" is the name of the component even if the
         name of the base package evolves with major release numbers
         (see "commons.osgi.symbolicName", below). -->
    <commons.componentid>math4</commons.componentid>
---CUT---
Correct or not?

It also uses a fix string:
---CUT---
<commons.scmPubUrl>https://svn.apache.org/repos/infra/websites/production/commons/content/proper/commons-math</commons.scmPubUrl>
---CUT---
whereas a variable (as in the commit below) would seem more portable.

Why isn't "<commons.scmPubUrl>" defined in CP only (using the appropriate
variable overridden in each component)?

Why having
---CUT---
<commons.site.path>math</commons.site.path>
---CUT---
that doesn't look at all like a "path"?
We could define a quite more explicit "<commons.component.name>"
that could serve for composing a path, as well as for any
other purpose where the component is meant, independently
of artefact identifier syntax or major version.


Regards,
Gilles

Gary

On Wed, Sep 19, 2018, 04:07 Gilles <gil...@harfang.homelinux.org> wrote:

Hi.

Are we sure that the fix/revert below to work as intended for
*all* components?

Common usage should be enforced (e.g. to allow anyone to help
releasing any component), and ancient inconsistencies fixed.

With a concrete example of a component that has had major
version changes (and top-level package change accordingly),
what is
  commons.componentid
and what is
  project.artefactId
?

Thanks,
Gilles

On Wed, 19 Sep 2018 08:06:17 -0000, brit...@apache.org wrote:
> Author: britter
> Date: Wed Sep 19 08:06:17 2018
> New Revision: 1841296
>
> URL: http://svn.apache.org/viewvc?rev=1841296&view=rev
> Log:
> COMMONSSITE-124: Revert change in commons.scmPubUrl in Parent 47
>
> Modified:
>     commons/proper/commons-parent/trunk/pom.xml
>     commons/proper/commons-parent/trunk/src/changes/changes.xml
>
> Modified: commons/proper/commons-parent/trunk/pom.xml
> URL:
>
>

http://svn.apache.org/viewvc/commons/proper/commons-parent/trunk/pom.xml?rev=1841296&r1=1841295&r2=1841296&view=diff
>
>

==============================================================================
> --- commons/proper/commons-parent/trunk/pom.xml (original)
> +++ commons/proper/commons-parent/trunk/pom.xml Wed Sep 19 08:06:17
> 2018
> @@ -1940,7 +1940,7 @@
>      <!-- value modules can override it -->
>      <commons.site.path>${commons.componentid}</commons.site.path>
>
> -
>
> <commons.scmPubUrl>

https://svn.apache.org/repos/infra/websites/production/commons/content/proper/${commons.componentid}
</commons.scmPubUrl>
> +
>
> <commons.scmPubUrl>

https://svn.apache.org/repos/infra/websites/production/commons/content/proper/${project.artifactId}
</commons.scmPubUrl>
>
>
>

<commons.scmPubCheckoutDirectory>${commons.site.cache}/${commons.site.path}</commons.scmPubCheckoutDirectory>
>      <commons.scmPubServer>commons.site</commons.scmPubServer>
>
>
> Modified: commons/proper/commons-parent/trunk/src/changes/changes.xml
> URL:
>
>

http://svn.apache.org/viewvc/commons/proper/commons-parent/trunk/src/changes/changes.xml?rev=1841296&r1=1841295&r2=1841296&view=diff
>
>

==============================================================================
> --- commons/proper/commons-parent/trunk/src/changes/changes.xml
> (original)
> +++ commons/proper/commons-parent/trunk/src/changes/changes.xml Wed
> Sep 19 08:06:17 2018
> @@ -61,6 +61,7 @@ The <action> type attribute can be add,u
>      <body>
>
> <release version="48" date="YYYY-MM-DD" description="TBD">
> +            <action issue="COMMONSSITE-124" type="fix"
> dev="britter">Revert change in commons.scmPubUrl in Parent
> 47</action>
>              <action type="update">Update parent from
> org.apache:apache 19 to 20.</action>
>              <action type="update">maven-compiler-plugin 3.7.0 ->
> 3.8.0</action>
>              <action type="fix">re-enabled japicmp and introduced
> japicmp.skip property that



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org




---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to