Paul Cunningham wrote:

>    Why isn't  --enable-webdav in the 64 bit configure as its
>    in the 32 bit ?

Fixed -- thank you for catching this.

> 2. usr/src/lib/libneon/install-sfw
>      & usr/src/lib/libneon/install-sfw-64
>    Don't deliver files with the write-permission bit set into /usr

OK, i can't do this either. Both libneon and subversion were 
originally integrated with the permissions set as you see them right 
now. If i change the permissions, there will be complaints from 
protocmp, and that will evolve into someone else complaining about the 
complaints from protocmp. Either way, someone will complain.

As it is right now, permissions bits remain identical with the 
previous (original) integration. That avoids pointless complaints at 
the bottom of nightly.log, and also avoids another round of 
conversations about why is protocmp complaining about file permission 
bits, when protocmp shouldn't be complaining about file permission bits.

And this is a very good illustration of why code reviews should 
perhaps avoid dealing with stylistic preferences which aren't really 
standardized anywhere, but are merely, and mostly, a matter of 
personal preference.

If there really was an interdiction about not delivering 0644 header 
files under /usr, then the previous integration would not have 
happened with the currently existing permissions, or it would have 
been corrected sometime between 2006 and now, or someone would at 
least have filed a CR about wrong permission bits for libneon and 
subversion header files. None of this happened.

I also see a whole bunch of header files with permission 0644 under 
/usr/include/*.h, and subdirectories.

> 4. SUNWneon/copyright
>    You have changed the licence in the METADATA, so should
>    this have changed also?

License has not changed, it was wrongly identified in METADATA. I 
corrected the license name in METADATA. libneon was not previously 
released under GPL V2, but LGPL V2, just as the current version is.

--Stefan

-- 
Stefan Teleman
Sun Microsystems, Inc.
stefan.teleman at Sun.COM

Reply via email to