<html> <head> <base href="https://bugs.documentfoundation.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - replace ScaList in scaddins with a std container" href="https://bugs.documentfoundation.org/show_bug.cgi?id=90222#c7">Comment # 7</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - replace ScaList in scaddins with a std container" href="https://bugs.documentfoundation.org/show_bug.cgi?id=90222">bug 90222</a> from <a class="email" href="mailto:michael.meeks@collabora.com" title="Michael Meeks <michael.meeks@collabora.com>"> Michael Meeks</a> <pre>Hi Pieter - there is no need to add a new class to the makefiles, but assuming you mean a new source module try: scaddins/Library_*.mk eg. $(eval $(call gb_Library_add_exception_objects,analysis,\ scaddins/source/analysis/analysis \ scaddins/source/analysis/analysishelper \ These guys are all paths to .cxx files eg. just add your path to the relevant libraries' Library_*.mk and you should be fine. Looking forward to reading the patch. If you do two un-related changes: eg. fixing the STL container issue, and then re-factoring copy/paste code - please submit these as separate commits to make the history comprehensible ! Good stuff :-)</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are on the CC list for the bug.</li> </ul> </body> </html>