On Thu, Feb 16, 2012 at 11:40, Thomas Mortagne <[email protected]>wrote:
> On Thu, Feb 16, 2012 at 11:07 AM, Caleb James DeLisle > <[email protected]> wrote: > > Hi, > > > > I'd like to switch filesystem attachments to begin using the persistent > storage directory now instead of the work directory. > > This means there's a new way of calculating where the attachments will > be stored so it might fail on upgrade. > > I would like to not do any migration and just add to the release notes > because: > > #1, it doesn't cause any permanent harm so long as nobody adds > attachments while it's in what is an obviously broken state. > > #2 administrators who have FS attachments enabled are probably going to > know what's going on. > > #3 migration code is scary, it requires lots of work and lots of review > and even if it works, > > people might feel violated having files shuffled around on their system > without their permission. > > > > WDYT? > > +1 and +1 for no migration by default too. If admin did his job > properly, work dir and permanent dir are supposed to be the same > I do not fully agree. My work dir has always been been stored in a place like /var/cache, where the data is not backup and has not real importance (I was not using filesystem attachement obviously). This was the default on a debian install. On the other hand, permanentDirectory should be in /var/lib, and be backed up. This is my setup right now with the patch of Caleb. > anyway and if the permanent dir is not set then admin just need to set > it and it OK. > > > > > Caleb > > > > _______________________________________________ > > devs mailing list > > [email protected] > > http://lists.xwiki.org/mailman/listinfo/devs > > > > -- > Thomas Mortagne > _______________________________________________ > devs mailing list > [email protected] > http://lists.xwiki.org/mailman/listinfo/devs > -- Denis Gervalle SOFTEC sa - CEO eGuilde sarl - CTO _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
