On Sun Dec 06 15:54:29 2015, tonyc wrote: > On Thu Oct 22 11:30:43 2015, dennis@kaarsemaker.net wrote: > > Having played a bit with travis, I think the patch could use some > > improvement. Attached is a patch which imho is a bit better: > > > > - Uses the container based builds and better dependencies > > - Doesn't skip eintr tests as it's not needed > > - Let travis build the build matrix based on os/compiler/env > > (consensus > > needed on useful ./Configure arguments > > - Set up notifications (definitely consensus needed before applying!) > > - Build on osx and linux > > - Only test blead/maint/smoke-me > > +# No testing of random branches, just blead, maint and smoke-me/* > +branches: > + only: > + - blead > + - /^maint-5.*$/ > + - /^smoke-me.*$/ > > This won't necessarily be used by perl5-porters members, it might be > used by someone outside who's preparing a patch to be submitted. > > I don't think we should expect such a user to name their branch to > match the above to get travis coverage. > > +# Minimal notifications (well, almost: email: false and leaving out > the irc > +# section would disable notifications completely) > +notifications: > + email: > + recipients: > + - perl5-porters@perl.org > + on_failure: change > + irc: > + channels: > + - "irc.perl.org#p5p" > + on_failure: change > > Similar to the above, the user might not know about the IRC channel > and might not be a member of the list. > > + # CC is exported by travis based on compuler section above > > "compiler". > > Tony Any progress on this? -- Karl Williamson --- via perlbug: queue: perl5 status: open https://rt.perl.org/Ticket/Display.html?id=123981Thread Previous