On 4 March 2016 at 15:40, Sam Ruby <ru...@intertwingly.net> wrote: > On Fri, Mar 4, 2016 at 10:03 AM, sebb <seb...@gmail.com> wrote: >> On 4 March 2016 at 00:19, Sam Ruby <ru...@intertwingly.net> wrote: >>> On Thu, Mar 3, 2016 at 4:35 PM, sebb <seb...@gmail.com> wrote: >>>> On 3 March 2016 at 21:12, Sam Ruby <ru...@intertwingly.net> wrote: >>>>> On Thu, Mar 3, 2016 at 3:50 PM, sebb <seb...@gmail.com> wrote: >>>>>> On 3 March 2016 at 13:28, Sam Ruby <ru...@intertwingly.net> wrote: >>>>>>> On Wed, Mar 2, 2016 at 9:22 AM, sebb <seb...@gmail.com> wrote: >>>>>>>> >>>>>>>> Better would be to have a function to report which entries are >>>>>>>> missing, so the user can decide which they want to download based on >>>>>>>> what they are testing. >>>>>>>> >>>>>>>> Maybe also with a function to report which are out of date. >>>>>>> >>>>>>> I've roughed in such a tool. You will need to do a "bundle update" or >>>>>>> otherwise get the latest Wunderbar (1.0.21) to test it out locally. >>>>>>> Then run: >>>>>>> >>>>>>> ruby www/status/svn.cgi >>>>>>> >>>>>>> And then visit http://localhost:4567/ >>>>>> >>>>>> That's very cool. >>>>>> >>>>>> I'd like to add the local path details if not in read-only mode. >>>>>> Would that be OK? >>>>> >>>>> Local path details are already there :-) >>>>> >>>>> Go to https://whimsy.apache.org/status/svn and hover over any text in >>>>> the first column. >>>> >>>> I see, that's useful. >>>> >>>> However one cannot copy the hover text nor search it except by looking >>>> at the source. >>> >>> OK, so how about a column that only shows if the window is wide enough? >>> >>> https://github.com/apache/whimsy/commit/c26dbfdd4b0a258b3fdbdb7f122d0da4e6b606c6 >> >> Great, though maybe it should be disabled for the read-only case i.e. >> on the live system. > > I don't see the harm.
No harm, but it's not particularly useful in that case. So it's better not to clutter the screen. >> Maybe it could depend on whether the script was run directly or via >> the webserver? >> >> If run under a webserver, one probably does not want the buttons either. > > I'm currently running it under a webserver with suexec enabled, and > the buttons work for me. That being said, if/when you try out > https://whimsy.apache.org/test/vhost-generator and it gets to the > point where it works for you, I plan to switch to that. The secretary > workbench is the only remaining application that updates files in the > svn working directory (something that has proven to be problematic), > and its eventual replacement (the secmail tool) will no longer do > that. So I won't need suexec any more. > > In any case, you don't need my permission to make updates. OK, thanks. But it's more a question of ensuring that I'm not overlooking other aspects of the app and committing unnecessary/conflicting changes. >>>>> I did it this way so that super-wide screens aren't required. >>>>> >>>>> - Sam Ruby >>> >>> - Sam Ruby > > - Sam Ruby
