Hi David, skimming through the 2.10.1-svn-18 commit, it seems it has entered on trunk. I recall that code being necessary so some test didn't create duplicate folders, but I'd rather check the svn history of that file to see where & why that replace was made. In any case, it should generate a folder inside target not in user.home or java.io.tmpdir, and should remove the digits only the autogenerated part of the path (not the user.home/java.io.tmpdir)
br, juan pablo On Tue, Jun 23, 2015 at 10:48 AM, David Vittor <[email protected]> wrote: > Funny that I had the same problem yesterday. > > I was trying to build the code on another laptop where my username was > C:/Users/david4 and the code kept removing the "4" after my name. > > I tracked this down in the testing code TestEngine.cleanTestProps(), and > here are my comments - not checked in :) > > props.setProperty( AbstractFileProvider.PROP_PAGEDIR, > pageDir // TODO: Is this really required, why > remove digits? .replaceAll( "\\d", StringUtils.EMPTY ) > + System.currentTimeMillis() ); > > Let me know if you want to remove this replaceAll, and check it in. > > Cheers, > David V > > > > > On Tue, Jun 23, 2015 at 6:09 PM, Juan Pablo Santos Rodríguez < > [email protected]> wrote: > > > Hi Dave, > > > > you should be able to run the build without any special permissions (that > > is, the files generated by the build should be under ./target/whatever), > > would you mind filing a JIRA so this doesn't get lost? > > > > > > thanks, > > juan pablo > > > > On Tue, Jun 23, 2015 at 9:58 AM, Dave Koelmeyer < > > [email protected]> wrote: > > > > > Hi All, > > > > > > On 23/06/15 18:32, Juan Pablo Santos Rodríguez wrote: > > > > > > > > I'll look closely into it this afternoon, but currently our CI build > is > > > > passing [#1], so it seems somehow related to the build environment? > > > > > > > > The offending test is trying to create a wiki page dir under > > > > $java.io.tmpir/non-existent-directory, which is the reason of the > > failing > > > > test. Dave, would you mind confirming that your user is able to > create > > > > files/folders inside that directory? In anycase, I'll switch the test > > > this > > > > afternoon so it creates the temporary folder inside ./target, in > order > > to > > > > avoid file-access permissions problems. > > > > > > My bad, running mvn pacakge as the superuser gives me a successful > build > > > (sorry, I should have tried this before posting). Curiously, assuming > my > > > login account is a non-root user, and is called "dkoe123", a new > > > directory is created as a byproduct of running sudo mvn package, called > > > "dkoe", and located at /home. Several directories deep, it contains > > > these folders (and is otherwise empty): > > > > > > /home/dkoe/Desktop/jspwiki/jspwiki-war/target > > > > > > Anyway thanks for the pointer :) > > > > > > Cheers, > > > Dave > > > > > > > > > > > > > juan pablo > > > > > > > > > > > > [#1]: https://builds.apache.org/job/JSPWiki/ > > > > > > > > On Tue, Jun 23, 2015 at 8:11 AM, Florian Holeczek < > [email protected] > > > > > > > wrote: > > > > > > > >> Hi Dave and Juan Pablo, > > > >> > > > >>> I'm attempting to build JSPWiki from source at the moment and Maven > > is > > > >>> failing with this error: > > > >> > > > >> seems to have been introduced with Juan Pablo's little commit: > > > >> > > > >> URL: http://svn.apache.org/r1684742 > > > >> Log: > > > >> change sample keys so they match with the ones at at > > > >> org.apache.wiki.attachment.AttachmentManager > > > >> > > > >> I didn't have a look at the details though, just confirming. > > > >> > > > >> Regards > > > >> Florian > > > > > > > > >
