Yep, that’s correct.

To be honest, I’m not entirely sure where the sitemap-sc.xml file came from. I think it may have been added to explicitly include the sitemapextension data somewhere, since they’re not applicable to the sitemapindex, and aren’t part of the actual sitemaps generated by our offline script.*

Based on more understanding of the codebase, today I updated the pull request to use a trigger handler in the repository configuration. That means it can be fixed up without any changes to the eprints core.

We’ll deploy something very much like it in our repository’s next release.

* It also hadn’t occurred to me that the script we use to generate our abstracts was written in-house (not part of EPrints proper), so we can change that, too. This will allow me to fix another issue we have (we’re currently generating a single sitemap XML file, instead of splitting it into separate files with up to 50_000 <url/>s in each.)

Cheers

Dear Matthew,

thanks for your response! Just to make it clear, we're talking about the same thing: http://eprints.qut.edu.au/sitemap-sc.xml actually doesn't have any <url/>-Tag ?!
Everything looks like @ our www.zora.uzh.ch/sitemap.xml

Cheers

--
Jens Vieler
Informatikdienste
Universität Zürich
Stampfenbachstrasse 73
CH-8006 Zürich

mail:  jens.vieler@id.uzh.ch
phone: +41 44 63 56777
http://www.id.uzh.ch

Matthew Kerwin ---15.12.2015 02:12:01---I’m pretty sure it was something my team added to our repository at some point, that got merged mid

Von: Matthew Kerwin <matthew.kerwin@qut.edu.au>
An: "eprints-tech@ecs.soton.ac.uk" <eprints-tech@ecs.soton.ac.uk>
Datum: 15.12.2015 02:12
Betreff: [EP-tech] Re: Sitemap changed?
Gesendet von: eprints-tech-bounces@ecs.soton.ac.uk

I’m pretty sure it was something my team added to our repository at some point, that got merged mid last year along with a bunch of our other customisations. (It has my name against the commit.) I’m pretty sure my PR (369) is just fixing a bug we introduced earlier.
 
Also having looked over it a few more times, I’m quite confident the new <url/> tag is inserted in the right place. Perhaps it would make more sense to use EP_TRIGGER_LOCAL_SITEMAP_URLS instead?
 
Cheers
--
Matthew Kerwin  |  QUT Library eServices  |  matthew.kerwin@qut.edu.au
 
From: eprints-tech-bounces@ecs.soton.ac.uk [mailto:eprints-tech-bounces@ecs.soton.ac.uk] On Behalf Of jens.vieler@id.uzh.ch
Sent: Tuesday, 15 December 2015 00:25
To: <eprints-tech@ecs.soton.ac.uk>
Subject: [EP-tech] Sitemap changed?
 

Dear List

i read https://github.com/eprints/eprints/pull/369 and although we cearte our sitemaps by ourselves, i tested them against google Webmaster Tools and got an error too.
Who changed the rules?

Jens

--
Jens Vieler
Informatikdienste
Universität Zürich
Stampfenbachstrasse 73
CH-8006 Zürich

mail:  jens.vieler@id.uzh.ch
phone: +41 44 63 56777
http://www.id.uzh.ch[Anhang "smime.p7s" gelöscht von Jens-Patrick Vieler/at/UZH] *** Options: http://mailman.ecs.soton.ac.uk/mailman/listinfo/eprints-tech
*** Archive: http://www.eprints.org/tech.php/
*** EPrints community wiki: http://wiki.eprints.org/
*** EPrints developers Forum: http://forum.eprints.org/