You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openoffice.apache.org by ro...@cms.zones.apache.org, --...@cms.zones.apache.org, te...@cms.zones.apache.org, no need to apply patch <an...@apache.org> on 2012/06/24 18:13:25 UTC

CMS diff: Apache OpenOffice (Incubating)

Clone URL (Committers only):
https://cms.apache.org/redirect?new=anonymous;action=diff;uri=http://openofficeorg.apache.org/openofficeorg%2Findex.mdtext

robweir -- testing, no need to apply patch

Index: /trunk/content/openofficeorg/index.mdtext
===================================================================
--- /trunk/content/openofficeorg/index.mdtext	(revision 1353213)
+++ /trunk/content/openofficeorg/index.mdtext	(working copy)
@@ -7,8 +7,8 @@
   - [Download Binaries](http://download.openoffice.org)
 
 Apache OpenOffice is comprised of six personal productivity applications: a word processor (and its web-authoring component), 
-spreadsheet, presentation graphics, drawing, equation editor, and database. OpenOffice is released on Windows, Solaris, Linux and 
-Macintosh operation systems, with more communities joining, including a mature FreeBSD port. OpenOffice is localized, supporting 
+spreadsheet, presentation graphics, drawing, equation editor, and database. OpenOffice is released on Windows, Linux and 
+Macintosh operation systems, with more ports in progress, including FreeBSD, Solaris and OS/2. OpenOffice is localized, supporting 
 over 110 languages worldwide. 
 
 OpenOffice.org  [was donated] [4] to the Apache Software Foundation (ASF) on June 1, 2011. As with any code base for consideration 


Re: CMS diff: Apache OpenOffice (Incubating)

Posted by Joe Schaefer <jo...@yahoo.com>.
Just a brief caution here: when you clone another user you
are creating an "unmanaged" dependency on their working copy tree,
which means if for some reason it becomes time to destroy
that working copy, yours will be destroyed as well.

For this reason it may be a good idea to "Force a New Working Copy"
once you are done with a cloned one, but unless you are doing lots
of bulk changes over an extended period of time you probably don't
need to bother with that (the CMS will reap working copies older than
3 days anyway).  In any case the "Force" links for that are availableat

https://cms.apache.org/$project/ or just https://cms.apache.org/.



----- Original Message -----
> From: Dave Fisher <da...@comcast.net>
> To: ooo-dev@incubator.apache.org
> Cc: 
> Sent: Sunday, June 24, 2012 12:59 PM
> Subject: Re: CMS diff: Apache OpenOffice (Incubating)
> 
> 
> On Jun 24, 2012, at 9:13 AM, robweir@cms.zones.apache.org, 
> --@cms.zones.apache.org, testing@cms.zones.apache.org, no need to apply patch 
> wrote:
> 
>>  Clone URL (Committers only):
>> 
> https://cms.apache.org/redirect?new=anonymous;action=diff;uri=http://openofficeorg.apache.org/openofficeorg%2Findex.mdtext
> 
> The link works for me. Very cool, I can clone Rob...
> 
> Regards,
> Dave
> 
>> 
>>  robweir -- testing, no need to apply patch
>> 
>>  Index: /trunk/content/openofficeorg/index.mdtext
>>  ===================================================================
>>  --- /trunk/content/openofficeorg/index.mdtext    (revision 1353213)
>>  +++ /trunk/content/openofficeorg/index.mdtext    (working copy)
>>  @@ -7,8 +7,8 @@
>>    - [Download Binaries](http://download.openoffice.org)
>> 
>>  Apache OpenOffice is comprised of six personal productivity applications: a 
> word processor (and its web-authoring component), 
>>  -spreadsheet, presentation graphics, drawing, equation editor, and 
> database. OpenOffice is released on Windows, Solaris, Linux and 
>>  -Macintosh operation systems, with more communities joining, including a 
> mature FreeBSD port. OpenOffice is localized, supporting 
>>  +spreadsheet, presentation graphics, drawing, equation editor, and 
> database. OpenOffice is released on Windows, Linux and 
>>  +Macintosh operation systems, with more ports in progress, including 
> FreeBSD, Solaris and OS/2. OpenOffice is localized, supporting 
>>  over 110 languages worldwide. 
>> 
>>  OpenOffice.org  [was donated] [4] to the Apache Software Foundation (ASF) 
> on June 1, 2011. As with any code base for consideration 
>> 
> 

Re: CMS diff: Apache OpenOffice (Incubating)

Posted by Dave Fisher <da...@comcast.net>.
On Jun 24, 2012, at 9:13 AM, robweir@cms.zones.apache.org, --@cms.zones.apache.org, testing@cms.zones.apache.org, no need to apply patch wrote:

> Clone URL (Committers only):
> https://cms.apache.org/redirect?new=anonymous;action=diff;uri=http://openofficeorg.apache.org/openofficeorg%2Findex.mdtext

The link works for me. Very cool, I can clone Rob...

Regards,
Dave

> 
> robweir -- testing, no need to apply patch
> 
> Index: /trunk/content/openofficeorg/index.mdtext
> ===================================================================
> --- /trunk/content/openofficeorg/index.mdtext	(revision 1353213)
> +++ /trunk/content/openofficeorg/index.mdtext	(working copy)
> @@ -7,8 +7,8 @@
>   - [Download Binaries](http://download.openoffice.org)
> 
> Apache OpenOffice is comprised of six personal productivity applications: a word processor (and its web-authoring component), 
> -spreadsheet, presentation graphics, drawing, equation editor, and database. OpenOffice is released on Windows, Solaris, Linux and 
> -Macintosh operation systems, with more communities joining, including a mature FreeBSD port. OpenOffice is localized, supporting 
> +spreadsheet, presentation graphics, drawing, equation editor, and database. OpenOffice is released on Windows, Linux and 
> +Macintosh operation systems, with more ports in progress, including FreeBSD, Solaris and OS/2. OpenOffice is localized, supporting 
> over 110 languages worldwide. 
> 
> OpenOffice.org  [was donated] [4] to the Apache Software Foundation (ASF) on June 1, 2011. As with any code base for consideration 
> 


Re: CMS diff: Apache OpenOffice (Incubating)

Posted by Joe Schaefer <jo...@yahoo.com>.
----- Original Message -----

> From: Rob Weir <ro...@apache.org>
> To: ooo-dev@incubator.apache.org
> Cc: 
> Sent: Sunday, June 24, 2012 12:21 PM
> Subject: Re: CMS diff: Apache OpenOffice (Incubating)
> 
> On Sun, Jun 24, 2012 at 12:13 PM,  <ro...@cms.zones.apache.org> wrote:
>>  Clone URL (Committers only):
>> 
> https://cms.apache.org/redirect?new=anonymous;action=diff;uri=http://openofficeorg.apache.org/openofficeorg%2Findex.mdtext
>> 
>>  robweir -- testing, no need to apply patch
>> 
> 
> So getting the patch generated and mailed worked.  But when I then
> went into my mail and tried to click the clone link, I got an error
> about not being able to clone myself.  I'm not an expert in bioethics,
> but this sounds like a reasonable prohibition ;-)
> 
> Exiting browser and going back in to Gmail fixed the problem.  I think
> the issue was the browser was caching the CMS authentication for user
> anonymous, and I had to clear that first.  But this is a weird
> scenario, one that would only come up in a demo.  Typically the person
> making and applying the patch are two different people.
> 
> So overall this looks cool.  Two enhancements that would make it even better:

Thanks!
 
> 1) Some static text saying something along the lines of "By clicking
> submit you agree to provide this patch under the terms of the Apache
> License 2.0 [link]"

Seems a bit bureaucratic for my tastes (personally loathe the jira checkbox),
but if others see a need perhaps I can do something about it.

> 
> 2) A text box for entering a comment associated with the patch, which
> would be inserted into the email.   This helps the committer applying
> the patch understand what they are looking at, if it is not obvious.

My response to that is to avoid using Quick Mail for such cases and just
submit your Edit => Diff => Mail Diff, which contains a full textarea for
patch commentary.  A good tutorial (hint) should discuss the distinction
between lightweight changes, for which Quick Mail is suitable, and those
more involved, where the "full procedure" is better suited for the task.

> 
> 
> -Rob
> 
>>  Index: /trunk/content/openofficeorg/index.mdtext
>>  ===================================================================
>>  --- /trunk/content/openofficeorg/index.mdtext   (revision 1353213)
>>  +++ /trunk/content/openofficeorg/index.mdtext   (working copy)
>>  @@ -7,8 +7,8 @@
>>    - [Download Binaries](http://download.openoffice.org)
>> 
>>   Apache OpenOffice is comprised of six personal productivity applications: 
> a word processor (and its web-authoring component),
>>  -spreadsheet, presentation graphics, drawing, equation editor, and 
> database. OpenOffice is released on Windows, Solaris, Linux and
>>  -Macintosh operation systems, with more communities joining, including a 
> mature FreeBSD port. OpenOffice is localized, supporting
>>  +spreadsheet, presentation graphics, drawing, equation editor, and 
> database. OpenOffice is released on Windows, Linux and
>>  +Macintosh operation systems, with more ports in progress, including 
> FreeBSD, Solaris and OS/2. OpenOffice is localized, supporting
>>   over 110 languages worldwide.
>> 
>>   OpenOffice.org  [was donated] [4] to the Apache Software Foundation (ASF) 
> on June 1, 2011. As with any code base for consideration
>> 
> 

Re: CMS diff: Apache OpenOffice (Incubating)

Posted by Rob Weir <ro...@apache.org>.
On Sun, Jun 24, 2012 at 12:13 PM,  <ro...@cms.zones.apache.org> wrote:
> Clone URL (Committers only):
> https://cms.apache.org/redirect?new=anonymous;action=diff;uri=http://openofficeorg.apache.org/openofficeorg%2Findex.mdtext
>
> robweir -- testing, no need to apply patch
>

So getting the patch generated and mailed worked.  But when I then
went into my mail and tried to click the clone link, I got an error
about not being able to clone myself.  I'm not an expert in bioethics,
but this sounds like a reasonable prohibition ;-)

Exiting browser and going back in to Gmail fixed the problem.  I think
the issue was the browser was caching the CMS authentication for user
anonymous, and I had to clear that first.  But this is a weird
scenario, one that would only come up in a demo.  Typically the person
making and applying the patch are two different people.

So overall this looks cool.  Two enhancements that would make it even better:

1) Some static text saying something along the lines of "By clicking
submit you agree to provide this patch under the terms of the Apache
License 2.0 [link]"

2) A text box for entering a comment associated with the patch, which
would be inserted into the email.   This helps the committer applying
the patch understand what they are looking at, if it is not obvious.


-Rob

> Index: /trunk/content/openofficeorg/index.mdtext
> ===================================================================
> --- /trunk/content/openofficeorg/index.mdtext   (revision 1353213)
> +++ /trunk/content/openofficeorg/index.mdtext   (working copy)
> @@ -7,8 +7,8 @@
>   - [Download Binaries](http://download.openoffice.org)
>
>  Apache OpenOffice is comprised of six personal productivity applications: a word processor (and its web-authoring component),
> -spreadsheet, presentation graphics, drawing, equation editor, and database. OpenOffice is released on Windows, Solaris, Linux and
> -Macintosh operation systems, with more communities joining, including a mature FreeBSD port. OpenOffice is localized, supporting
> +spreadsheet, presentation graphics, drawing, equation editor, and database. OpenOffice is released on Windows, Linux and
> +Macintosh operation systems, with more ports in progress, including FreeBSD, Solaris and OS/2. OpenOffice is localized, supporting
>  over 110 languages worldwide.
>
>  OpenOffice.org  [was donated] [4] to the Apache Software Foundation (ASF) on June 1, 2011. As with any code base for consideration
>