Passed all the tests on Travis! Which was why I thought the docs used a different set of code.
Ian On Wed, 11 Dec 2019, 09:04 Andrea Aime, <andrea.a...@geo-solutions.it> wrote: > Hi, > did the change go through a PR? The travis build should fail when docs > fail to build > > Cheers > Andrea > > > On Wed, Dec 11, 2019 at 10:01 AM Ian Turton <ijtur...@gmail.com> wrote: > >> Sorry, I thought the docs were running off a separate copy of the code. >> >> Ian >> >> On Wed, 11 Dec 2019 at 08:23, Jody Garnett <jody.garn...@gmail.com> >> wrote: >> >>> Looks to be: >>> >>> [exec] Warning, treated as error: >>> [exec] >>> /tmp/jenkins-e86d80c8/workspace/geotools-master-docs/docs/user/tutorial/datastore/strategy.rst:142:start-after >>> pattern not found: import com.csvreader.CsvWriter; >>> >>> >>> I think Ian made some changes to CSV datastore (to use a supported library) >>> and misplaced some comments used by the docs to inline code). >>> >>> -- >>> Jody Garnett >>> >>> >>> On Wed, 11 Dec 2019 at 09:21, Jody Garnett <jody.garn...@gmail.com> >>> wrote: >>> >>>> Having a look, we should version control this script. >>>> -- >>>> Jody Garnett >>>> >>>> >>>> On Tue, 10 Dec 2019 at 15:27, Andrea Aime <andrea.a...@geo-solutions.it> >>>> wrote: >>>> >>>>> See: >>>>> https://build.geoserver.org/job/geotools-master-docs/2622/consoleFull >>>>> >>>>> Looks like the build has been failing for a while, and it's likely >>>>> related to the build script >>>>> itself, as building the docs locally seems to work just fine. >>>>> Looking at the Jenkins configuration (has a bash script) the build >>>>> seems to be trying to >>>>> get into "target/web/html", which I have locally, but don't see in the >>>>> online working area: >>>>> https://build.geoserver.org/job/geotools-master-docs/ws/docs/target/ >>>>> >>>>> Maybe this instruction chaining prevents it from seeing it's not here? >>>>> cd docs && >>>>> mvn install && >>>>> cd target/web/html && >>>>> if [ -e web.zip ]; then >>>>> rm -rf web.zip >>>>> fi >>>>> >>>>> And then it fails trying to cd ../../target/use/html because it did >>>>> not get into target/web/html to start with? Unsure. Anyone has a clue? >>>>> >>>>> Cheers >>>>> Andrea >>>>> >>>>> -- >>>>> >>>>> Regards, Andrea Aime == GeoServer Professional Services from the >>>>> experts! Visit http://goo.gl/it488V for more information. == Ing. >>>>> Andrea Aime @geowolf Technical Lead GeoSolutions S.A.S. Via di Montramito >>>>> 3/A 55054 Massarosa (LU) phone: +39 0584 962313 fax: +39 0584 1660272 mob: >>>>> +39 339 8844549 http://www.geo-solutions.it >>>>> http://twitter.com/geosolutions_it >>>>> ------------------------------------------------------- *Con >>>>> riferimento alla normativa sul trattamento dei dati personali (Reg. UE >>>>> 2016/679 - Regolamento generale sulla protezione dei dati “GDPR”), si >>>>> precisa che ogni circostanza inerente alla presente email (il suo >>>>> contenuto, gli eventuali allegati, etc.) è un dato la cui conoscenza è >>>>> riservata al/i solo/i destinatario/i indicati dallo scrivente. Se il >>>>> messaggio Le è giunto per errore, è tenuta/o a cancellarlo, ogni altra >>>>> operazione è illecita. Le sarei comunque grato se potesse darmene notizia. >>>>> This email is intended only for the person or entity to which it is >>>>> addressed and may contain information that is privileged, confidential or >>>>> otherwise protected from disclosure. We remind that - as provided by >>>>> European Regulation 2016/679 “GDPR” - copying, dissemination or use of >>>>> this >>>>> e-mail or the information herein by anyone other than the intended >>>>> recipient is prohibited. If you have received this email by mistake, >>>>> please >>>>> notify us immediately by telephone or e-mail.* >>>>> _______________________________________________ >>>>> GeoTools-Devel mailing list >>>>> GeoTools-Devel@lists.sourceforge.net >>>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel >>>>> >>>> >> >> -- >> Ian Turton >> > > > -- > > Regards, Andrea Aime == GeoServer Professional Services from the experts! > Visit http://goo.gl/it488V for more information. == Ing. Andrea Aime > @geowolf Technical Lead GeoSolutions S.A.S. Via di Montramito 3/A 55054 > Massarosa (LU) phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 339 > 8844549 http://www.geo-solutions.it http://twitter.com/geosolutions_it > ------------------------------------------------------- *Con riferimento > alla normativa sul trattamento dei dati personali (Reg. UE 2016/679 - > Regolamento generale sulla protezione dei dati “GDPR”), si precisa che ogni > circostanza inerente alla presente email (il suo contenuto, gli eventuali > allegati, etc.) è un dato la cui conoscenza è riservata al/i solo/i > destinatario/i indicati dallo scrivente. Se il messaggio Le è giunto per > errore, è tenuta/o a cancellarlo, ogni altra operazione è illecita. Le > sarei comunque grato se potesse darmene notizia. This email is intended > only for the person or entity to which it is addressed and may contain > information that is privileged, confidential or otherwise protected from > disclosure. We remind that - as provided by European Regulation 2016/679 > “GDPR” - copying, dissemination or use of this e-mail or the information > herein by anyone other than the intended recipient is prohibited. If you > have received this email by mistake, please notify us immediately by > telephone or e-mail.* >
_______________________________________________ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
