<div dir="ltr">Hey Gurung,<br><br><div class="gmail_quote"><div dir="ltr">On Fri, Aug 24, 2018 at 6:35 PM Rahul Gurung <<a href="mailto:gurungrahul2@gmail.com">gurungrahul2@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hey,</div><div><br></div><div><br></div><div>I did the push, waiting to get it reviewed.</div></div></blockquote><div><br></div><div>I just pushed a few fixes for the patch. You removed quite a few files from the makefile which later breaks the build. Additionally, you forgot a ; after CPPUNIT_TEST which caused some additional errors.</div><div><br></div><div>An unrelated thing that I would suggest for the future is to not push patches as drafts as only admins and the people added by you as reviewers will see the change. You miss all the helpful hands of the remaining LibreOffice developers if they can not see your patch :)</div><div><br></div><div>Regards,</div><div>Markus</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>Regards,</div><div>Gurung<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 22 August 2018 at 23:11, Markus Mohrhard <span dir="ltr"><<a href="mailto:markus.mohrhard@googlemail.com" target="_blank">markus.mohrhard@googlemail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hey Rahul,</div><div><br></div><div>can you push your current state to gerrit so that we can discuss the code more easily?</div><div><br></div><div>Regards,</div><div>Markus<br></div></div><br><div class="gmail_quote"><div><div class="m_-4177846484100535608h5"><div dir="ltr">On Wed, Aug 22, 2018 at 7:28 PM Rahul Gurung <<a href="mailto:gurungrahul2@gmail.com" target="_blank">gurungrahul2@gmail.com</a>> wrote:<br></div></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="m_-4177846484100535608h5"><div dir="ltr"><div>Hey,</div><div><br></div><div>I was experimenting with cpp test by making a new test( transfered from java tests) and activating it when I run make test it threw this error, what can this error possibly mean, I know there can be many cases and this question may be unanswerable, but please help if possible.<br></div><div><br></div><div><br></div><div>/home/gurungrahul2/libreoffice/workdir/CxxObject/test/source/mtfxmldump.o: In function `MetafileXmlDump::dumpAndParse(GDIMetaFile const&, rtl::OUString const&)':<br>mtfxmldump.cxx:(.text+0x5868): undefined reference to `tools::XmlWriter::startDocument(int)'<br>/home/gurungrahul2/libreoffice/workdir/CxxObject/test/source/primitive2dxmldump.o: In function `Primitive2dXmlDump::dumpAndParse(drawinglayer::primitive2d::Primitive2DContainer const&, rtl::OUString const&)':<br>primitive2dxmldump.cxx:(.text+0x1608): undefined reference to `tools::XmlWriter::startDocument(int)'<br>/home/gurungrahul2/libreoffice/workdir/CxxObject/test/source/screenshot_test.o: In function `ScreenshotTest::setUp()':<br>screenshot_test.cxx:(.text+0x6a2): undefined reference to `test::Directories::getURLFromWorkdir(rtl::OUString const&) const'<br>/home/gurungrahul2/libreoffice/workdir/CxxObject/test/source/screenshot_test.o: In function `ScreenshotTest::implSaveScreenshot(BitmapEx const&, rtl::OString const&)':<br>screenshot_test.cxx:(.text+0xc68): undefined reference to `test::Directories::getURLFromWorkdir(rtl::OUString const&) const'<br>/home/gurungrahul2/libreoffice/workdir/CxxObject/test/source/screenshot_test.o: In function `ScreenshotTest::processDialogBatchFile(rtl::OUString const&)':<br>screenshot_test.cxx:(.text+0x2364): undefined reference to `test::Directories::getURLFromSrc(rtl::OUString const&) const'<br>collect2: error: ld returned 1 exit status<br>/home/gurungrahul2/libreoffice/solenv/gbuild/LinkTarget.mk:636: recipe for target '/home/gurungrahul2/libreoffice/workdir/LinkTarget/Library/libtest.so' failed<br>make[1]: *** [/home/gurungrahul2/libreoffice/workdir/LinkTarget/Library/libtest.so] Error 1<br>Makefile:120: recipe for target 'test' failed<br>make: *** [test] Error 2</div><div><br></div><div><br></div><div><br></div><div><br></div><div>Thanks.<br></div></div></div></div>
_______________________________________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org" target="_blank">LibreOffice@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/libreoffice" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
</blockquote></div>
</blockquote></div><br></div>
</blockquote></div></div>