Code review comment for lp:~weyrick/bzr/54624-warn-on-large-files

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Should this be this close to the core? I wonder if it makes more sense as a UI-level thing that's just part of "bzr add".

With the current approach, we'd have to introduce workarounds for things like the UDD importer, but I presume there will also be a fair number of packages for which "bzr merge-upstream" will now fail.

It also means all custom tree implementations will have to explicitly add support.

« Back to merge proposal