I am fixing now -- Jody Garnett
On Wed, 11 Dec 2019 at 10:30, Ian Turton <ijtur...@gmail.com> wrote: > 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
