Hi Lubos, I will just ask the legal team for the contributor agreement. Meanwhile, since the patches are public now I hope they will be useful to other users while we get the paperwork handled :)
cheers, -deniz On Thu, Dec 20, 2012 at 4:45 PM, Lubos Kosco <[email protected]> wrote: > > Oh yes, and thank you, I am happy we have so many people that take care > about {OpenGrok > (and I hope we can somehow expedite the patches processing) > > thnx > L > > > On 20.12.2012 16:44, Lubos Kosco wrote: >> >> >> Deniz >> >> to accept the patches under your name we would need an OCA # from you >> you can get it by following this process: >> http://hub.opensolaris.org/bin/view/Main/oca_faq >> (it's just a plain email exchange, takes 2-3 days most, they will register >> you as contributor and assign a #) >> >> the other way is that we will push it and just thank you in comments in >> case you give away the rights to your code (since those are small patches >> and not sure if they will be taken as they are - Knut, Trond ? ) >> >> and yes, I know it's PITA and I hope we will find a better way (Trond, >> how's the git history rewrite going on? ;) ) >> >> your choice >> Lubos >> >> On 20.12.2012 16:34, Deniz Türkoglu wrote: >>> >>> OpenGrok prints the exception when it fails to find a file that >>> makes the results page quite 'dirty'. This patch checks for the >>> file's existence and skips it if it doesn't exist. >>> >>> Signed-off-by: Deniz Turkoglu <[email protected]> >>> --- >>> src/org/opensolaris/opengrok/search/Results.java | 7 ++++--- >>> 1 file changed, 4 insertions(+), 3 deletions(-) >>> >>> diff --git a/src/org/opensolaris/opengrok/search/Results.java >>> b/src/org/opensolaris/opengrok/search/Results.java >>> index 7e0a0ae..8057bd3 100644 >>> --- a/src/org/opensolaris/opengrok/search/Results.java >>> +++ b/src/org/opensolaris/opengrok/search/Results.java >>> @@ -207,9 +207,10 @@ public final class Results { >>> String htags = getTags(sh.sourceRoot, rpath, >>> false); >>> out.write(sh.summerizer.getSummary(htags).toString()); >>> } else { >>> - FileReader r = genre == Genre.PLAIN >>> - ? new FileReader(new >>> File(sh.sourceRoot, rpath)) >>> - : null; >>> + File file = new File(sh.sourceRoot, rpath); >>> + FileReader r = >>> + (genre == Genre.PLAIN && file.exists()) ? >>> + new FileReader(file) : null; >>> sh.sourceContext.getContext(r, out, xrefPrefix, >>> morePrefix, rpath, tags, true, null); >>> } >>> -- >>> 1.8.0.2 >>> _______________________________________________ >>> opengrok-dev mailing list >>> [email protected] >>> http://mail.opensolaris.org/mailman/listinfo/opengrok-dev >> >> > _______________________________________________ opengrok-dev mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/opengrok-dev
