Re: [whimsy] branch master updated: Add dryrun to multiUpdate

2020-06-08 Thread sebb
The authentication is provided by the 'env' parameter. This must support env.user and env.password queries. It's rather difficult to advise further without sight of the full code. On Sun, 7 Jun 2020 at 01:20, Craig Russell wrote: > > Next problem should be easy to fix. Looks like there is some a

Re: [whimsy] branch master updated: Add dryrun to multiUpdate

2020-06-06 Thread Craig Russell
Next problem should be easy to fix. Looks like there is some authentication needed here: { "transcript": [ "$ svn checkout --depth empty --non-interactive --no-auth-cache https://svn.apache.org/repos/private/foundation /var/folders/mx/t44d3_bs437dpnywzk_mwl_0002msd/T/d20200606-97419-zmn43o

Re: [whimsy] branch master updated: Add dryrun to multiUpdate

2020-06-06 Thread Sam Ruby
On Sat, Jun 6, 2020 at 4:41 PM Craig Russell wrote: > > I tried this: > line 22: > ASF::SVN.multiUpdate (members_txt, message, env, _) do |text| delete the space between multiUpdate and the open parenthesis. In Ruby, parenthesis are generally optional on method calls. With the space after the m

Re: [whimsy] branch master updated: Add dryrun to multiUpdate

2020-06-06 Thread Craig Russell
FYI: The line 22 that finally worked: # update members.txt only for secretary actions ASF::SVN.multiUpdate members_txt, message, env, _ do |text| I don't know why having the () around the arguments is a problem, but it is. Could this be a bug in the "ruby compiler"? Craig > On Jun 6, 2020, at 1

Re: [whimsy] branch master updated: Add dryrun to multiUpdate

2020-06-06 Thread Craig Russell
> On Jun 6, 2020, at 3:32 PM, sebb wrote: > > On Sat, 6 Jun 2020 at 21:41, Craig Russell > wrote: >> >> I tried this: >> line 22: >> ASF::SVN.multiUpdate (members_txt, message, env, _) do |text| >> # default command is empty >> command = "" >> # remove user's e

Re: [whimsy] branch master updated: Add dryrun to multiUpdate

2020-06-06 Thread sebb
On Sat, 6 Jun 2020 at 21:41, Craig Russell wrote: > > I tried this: > line 22: > ASF::SVN.multiUpdate (members_txt, message, env, _) do |text| > # default command is empty > command = "" > # remove user's entry > unless text.sub! entry, '' # e.g. if the workspace was out of date > rais

Re: [whimsy] branch master updated: Add dryrun to multiUpdate

2020-06-06 Thread Craig Russell
I tried this: line 22: ASF::SVN.multiUpdate (members_txt, message, env, _) do |text| # default command is empty command = "" # remove user's entry unless text.sub! entry, '' # e.g. if the workspace was out of date raise Exception.new("Failed to remove existing entry -- try refreshing")

Re: [whimsy] branch master updated: Add dryrun to multiUpdate

2020-06-06 Thread sebb
The full method name is ASF::SVN.multiUpdate There's an example call here: https://lists.apache.org/thread.html/r97d0fc64bf633e643889cba21e18b5bb60c7d068430c36c447119887%40%3Cdev.whimsical.apache.org%3E On Sat, 6 Jun 2020 at 15:47, Craig Russell wrote: > > Is there an example of the use of thi

Re: [whimsy] branch master updated: Add dryrun to multiUpdate

2020-06-06 Thread Craig Russell
Is there an example of the use of this method? I cannot seem to call it from the memstat.json.rb code. Thanks, Craig > On Jun 6, 2020, at 7:18 AM, s...@apache.org wrote: > > This is an automated email from the ASF dual-hosted git repository. > > sebb pushed a commit to branch master > in repo