https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-12527 Lustre <p>If one tries to compile a C++ program with the `lustre/lustreapi.h` header, the compiler will throw all kinds of warnings. </p> <p>This issue came up on the mailing list a few years ago: <a href="http://lists.lustre.org/pipermail/lustre-discuss-lustre.org/2017-March/014268.html" class="external-link" target="_blank" rel="nofollow noopener">http://lists.lustre.org/pipermail/lustre-discuss-lustre.org/2017-March/014268.html</a> and I encountered it again in a different context this week. Guess not many C++ programs try to use lustreapi.h !</p> C++ LU-12527

lustreapi.h not c++ compatible

Improvement Minor Resolved Fixed Robert Latham Robert Latham Tue, 9 Jul 2019 21:03:05 +0000 Thu, 15 Aug 2019 13:44:28 +0000 Thu, 15 Aug 2019 13:44:28 +0000 Lustre 2.13.0 0 4 <p>I attached a small C++ program to demonstrate the error. Both `clang version 6.0.0-1ubuntu2 (tags/RELEASE_600/final)` and `g++ (Ubuntu 7.4.0-1ubuntu1~18.04.1) 7.4.0` give errors. </p> <p>Here's a rough patch that makes g++ and clang++ happy. </p> <p>Rob</p> <p>Thank you for the bug report and patch. Are you able to submit your patch directly to Gerrit (see <a href="http://wiki.lustre.org/Using_Gerrit" class="external-link" target="_blank" rel="nofollow noopener">http://wiki.lustre.org/Using_Gerrit</a> for details)? If not, then we can assign this ticket to another developer, but it will likely take longer to get attention</p> <p>Peter</p> <p>I'm going through the process. info@whamcloud probably gets a ton of spam, but it also got a request from me asking for permission to push to gerrit.</p> <p>Try now Rob</p> <p>Rob Latham (robl@mcs.anl.gov) uploaded a new patch: <a href="https://review.whamcloud.com/35471" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35471</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12527" title="lustreapi.h not c++ compatible" class="issue-link" data-issue-key="LU-12527"><del>LU-12527</del></a> utils: Make lustre_user.h c++-legal<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: e23c74817c7d1d2d2993e6a55bd4a899bfca342d</p> <p><a href="https://review.whamcloud.com/#/c/35471/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/#/c/35471/</a></p> <p>Oleg Drokin (green@whamcloud.com) merged in patch <a href="https://review.whamcloud.com/35471/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/35471/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-12527" title="lustreapi.h not c++ compatible" class="issue-link" data-issue-key="LU-12527"><del>LU-12527</del></a> utils: Make lustre_user.h c++-legal<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 14b11dc3526ad12a138ad750ab8824b3438dc00d</p> <p>Landed for 2.13 - congrats Rob!</p> Development Rank 1|i00jef: Rank (Obsolete) 9223372036854775807