Re: [commits] Horde branch master updated. fbb3a95b46589570ff613e597aa3d6225c8adb11
Michael J Rubinsky <[email protected]> Wed, 07 Sep 2016 17:26:58 +0000
| Newsgroups | gmane.comp.horde.devel |
|---|---|
| Message-ID | <20160907172658.Horde.0wZJV18NM5__80QgzqmqQ26@h4.theupstairsroom.com> |
Quoting Jan Schneider <[email protected]>: > Zitat von Michael J Rubinsky <[email protected]>: > >> Quoting Jan Schneider <[email protected]>: >> >>> The branch "master" has been updated. >>> The following is a summary of the commits. >>> >>> from: 811f94975a70923fa038f162f1907626d9038a76 >>> >>> fbb3a95 Update METAR station list (again?) >>> >>> Summary: >>> http://github.com/horde/horde/compare/811f94975a70923fa038f162f1907626d9038a76...fbb3a95b46589570ff613e597aa3d6225c8adb11 >>> >>> ----------------------------------------------------------------------- >>> >>> commit fbb3a95b46589570ff613e597aa3d6225c8adb11 >>> Author: Jan Schneider <[email protected]> >>> Date: Wed Sep 7 16:38:33 2016 +0200 >>> >>> Update METAR station list (again?) >>> >>> We should provide an alternative way to populate the list. This >>> may fail for other reasons like network problems too, and there's >>> no way to fill the database again, other than migrating down and up. >> >> That's why I added the check to see if the file existed locally, in >> the migration directory. Still requires documentation I guess, but >> manually downloading the file and placing it in the migration >> directory *should* work. > > Yes, I know, but you may only find out after the original migration > failed. The schema is still updated though at this point. So either > the migration should fail completely (via transactions for example), > or we should a horde-service-weather-populate script that uses the > then public populateTable() method. Ah, OK. I see. I would rather not *require* the execution of another script i.e. horde-service-weather-populate, to fill the table so I'll wrap the migration in a transaction. It probably wouldn't hurt to include a script that can manually (re)populate the table too in the case the list is updated. > -- > Jan Schneider > The Horde Project > http://www.horde.org/ > > -- > dev mailing list > Frequently Asked Questions: http://wiki.horde.org/FAQ > To unsubscribe, mail: [email protected] -- mike The Horde Project http://www.horde.org https://www.facebook.com/hordeproject https://www.twitter.com/hordeproject -- dev mailing list Frequently Asked Questions: http://wiki.horde.org/FAQ To unsubscribe, mail: [email protected]
smime.p7s
(application/pkcs7-signature, 5.6 KB) - not displayed