Hi Paul, Yes, the url and the license are correct. I was able to open the url today. Seems to be some issue yesterday on opening the url.
Rest of the changes are done and webrev uploaded at http://cr.opensolaris.org/~vivekrt/6835986-slf4j/ . Let me know if I am missing any. Thanks, ~Vivek R. Titarmare -----Original Message----- From: Paul Cunningham [mailto:[email protected]] Sent: Wednesday, May 27, 2009 6:00 PM To: Vivek Titarmare Cc: sfwnv-discuss at opensolaris.org Subject: Re: [sfwnv-discuss] Request code review for "slf4j" Paul Cunningham wrote: > This mainly looks okay to me, see below for a comments ... I missed a bit ... > > Vivek Titarmare wrote: >> I have posted a webrev for package "slf4j" which I am porting to >> Nevada and would like to request a code review. Please see the link below >> >> http://cr.opensolaris.org/~vivekrt/6835986-slf4j/ >> > > 1. usr/src/lib/Makefile > Why are there now two 'FRC:' rules ? > 164 > 165 FRC: > 166 ore > 167 > 168 FRC: > > 2. usr/src/pkgdefs/Makefile > Add the new entry alphabetically > > 3. usr/src/pkgdefs/SUNWslf4j/copyright > & usr/src/lib/slf4j/METADATA > Is the licence correct here ? 4. usr/src/lib/slf4j/METADATA Is the url "http://www.slf4j.org"; correct - it couldn't find it when I tried? -- ---------------------------------------------------------------------- Paul Cunningham Software Engineer Tadpole Business Unit
