Re: svn commit: r1792641 - /commons/proper/jexl/trunk/pom.xml

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1792641 - /commons/proper/jexl/trunk/pom.xml

Emmanuel Bourg-3
Le 25/04/2017 à 20:36, [hidden email] a écrit :

> --- commons/proper/jexl/trunk/pom.xml (original)
> +++ commons/proper/jexl/trunk/pom.xml Tue Apr 25 18:36:28 2017
> @@ -28,7 +28,7 @@
>      <name>Apache Commons JEXL</name>
>      <inceptionYear>2001</inceptionYear>
>      <description>The Apache Commons JEXL library is an implementation of the JSTL Expression Language with extensions.</description>
> -    <url>http://commons.apache.org/jexl/</url>
> +    <url>http://commons.apache.org/proper/commons-jexl/</url>

I'd prefer keeping the short form. I still hope we'll be able to get rid
of these horribly long URLs and return to the ones we had originally.

Emmanuel Bourg


---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1792641 - /commons/proper/jexl/trunk/pom.xml

sebb-2-2
On 25 April 2017 at 19:50, Emmanuel Bourg <[hidden email]> wrote:

> Le 25/04/2017 à 20:36, [hidden email] a écrit :
>
>> --- commons/proper/jexl/trunk/pom.xml (original)
>> +++ commons/proper/jexl/trunk/pom.xml Tue Apr 25 18:36:28 2017
>> @@ -28,7 +28,7 @@
>>      <name>Apache Commons JEXL</name>
>>      <inceptionYear>2001</inceptionYear>
>>      <description>The Apache Commons JEXL library is an implementation of the JSTL Expression Language with extensions.</description>
>> -    <url>http://commons.apache.org/jexl/</url>
>> +    <url>http://commons.apache.org/proper/commons-jexl/</url>
>
> I'd prefer keeping the short form.

Problem is that the short form does not generate the correct relative
links from component pages to the Commons parent site.
Such links need an extra ../ segment.
I'm just in the process of manually fixing JEXL.

> I still hope we'll be able to get rid
> of these horribly long URLs and return to the ones we had originally.

That's a separate issue.
Please start a new thread.

> Emmanuel Bourg
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1792641 - /commons/proper/jexl/trunk/pom.xml

Emmanuel Bourg-3
Le 25/04/2017 à 21:20, sebb a écrit :

> Problem is that the short form does not generate the correct relative
> links from component pages to the Commons parent site.
> Such links need an extra ../ segment.

Where are these links generated? We should switch them to absolute URLs.

Emmanuel Bourg


---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1792641 - /commons/proper/jexl/trunk/pom.xml

sebb-2-2
On 25 April 2017 at 20:36, Emmanuel Bourg <[hidden email]> wrote:
> Le 25/04/2017 à 21:20, sebb a écrit :
>
>> Problem is that the short form does not generate the correct relative
>> links from component pages to the Commons parent site.
>> Such links need an extra ../ segment.
>
> Where are these links generated? We should switch them to absolute URLs.

Why?

They are created currently so long as the correct <url> is used in the POM.

It took a lot of work to get the URLs working, as Maven does funny
things with URLs, even absolute ones.

And we want the links to work regardless of whether http/https is used.

> Emmanuel Bourg
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1792641 - /commons/proper/jexl/trunk/pom.xml

sebb-2-2
On 25 April 2017 at 22:35, sebb <[hidden email]> wrote:

> On 25 April 2017 at 20:36, Emmanuel Bourg <[hidden email]> wrote:
>> Le 25/04/2017 à 21:20, sebb a écrit :
>>
>>> Problem is that the short form does not generate the correct relative
>>> links from component pages to the Commons parent site.
>>> Such links need an extra ../ segment.
>>
>> Where are these links generated? We should switch them to absolute URLs.
>
> Why?
>
> They are created currently so long as the correct <url> is used in the POM.

They are created correctly so long as the correct <url> is used in the POM.

[Note that the <project><url> in the POM does not affect where the
site is deployed.
It merely records where the site will be found so the correct relative
links can be generated.]

> It took a lot of work to get the URLs working, as Maven does funny
> things with URLs, even absolute ones.
>
> And we want the links to work regardless of whether http/https is used.
>
>> Emmanuel Bourg
>>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]