RFR: JDK-8260966 (fs) Consolidate Linux and macOS implementations of UserDefinedFileAttributeView [v3]

Sebastian Stenzel sebastian.stenzel at gmail.com
Wed Mar 3 07:33:28 UTC 2021


> On 3. Mar 2021, at 08:23, Thomas Stuefe <stuefe at openjdk.java.net> wrote:
> 
> On Wed, 3 Mar 2021 06:51:27 GMT, Thomas Stuefe <stuefe at openjdk.org <mailto:stuefe at openjdk.org>> wrote:
> 
>>>> Just out of curiosity, since I only see the diff in GitHub or via my local git client: Are these webrev things somehow related to the review process and mess this up? In an earlier comment, brian suggested that individual commits in git are somehow "squashed" by webrev, causing all the review pain in the original PR in the first place.
>>>> 
>>>> I know this is off-topic, but I just want to understand what things I need to be aware of, if I contribute further patches. I have the feeling that Skara wasn't so much about transitioning to git and GitHub but rather added a bridge between those and some internal tooling.
>>> 
>>> Some people look at the diffs in on github where it's possible to customize the view. Some people look use the generated webrev. There's a bot that generates the webrev and adds a link to the PR, look for the comment from "mlbridge" ("mailing list bridge").
>> 
>> Unfortunately this broke AIX and I don't know if this would work on BSD (not sure if anyone still maintains those).
>> 
>> https://bugs.openjdk.java.net/browse/JDK-8262926 <https://bugs.openjdk.java.net/browse/JDK-8262926>
> 
> If you do a PR, I can test it for you.

Here you go: https://github.com/openjdk/jdk/pull/2805 <https://github.com/openjdk/jdk/pull/2805>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/nio-dev/attachments/20210303/6b7ee68a/attachment.htm>


More information about the nio-dev mailing list