Hi Robert, Stefan, On Tue, Mar 26, 2019 at 3:37 PM Robert J. Clay <rjclay@gmail.com> wrote:
There's not that much of a dojo related version difference with what LedgerSMB 1.6 uses (Debian testing has v1.14.2) but that's not the case for LSMB 1.7 so it's something I need to look at in any case.
Actually, as I read that sentence, you're saying that LedgerSMB 1.7 doesn't use Dojo 1.14.2 (and depending on how you read it, that LedgerSMB 1.6 does). However, that's incorrect: It's LedgerSMB 1.7 (which isn't currently in Debian Testing/Buster) uses 1.14.2, but LedgerSMB 1.6 - which *is* in Testing - uses Dojo 1.13. To be honest, I see that Dojo 1.15.0 has been released and I'll likely at least switch master over to that. (I think it's too close to the 1.7 release to change the Dojo library version right now.)
Actually I don't think that is a good idea. The release cycle starts
again after buster is ready, and there is enough time to get dojo in sync with what is in LedgerSMB.
Well, the problem is that in the past I've seen different versions of 1.x Dojo requiring adjustments in our code base to support Dojo version upgrades or downgrades. As a result, we advise to run LedgerSMB against the Dojo version it comes with, or optionally any other patch-level version, but *not* any other minor-level version, not even when that version is higher than the minimum required.
Or at least, with the dojo toolkit itself. A reason why LSMB includes it is because that version can get of sync, pariticularly with released LSMB versions, which track the major/minor version of dojo from at the time that the LSMB version was released.but might get the patch level version number sync'ed occasionally. (This is done using sub-modules in the LSMB code repository against the dojo code repository.)
Yes. Patch-level version differences are ok, although I've seen patch-level releases break functionality LedgerSMB depends on (unintentionally, with follow-up patch level releases fixing it again). All in all, the Dojo release being used seems somewhat fragile and needs thorough testing before the combination is geing changed.
Also it is the general policy in Debian to
use the available packaged version instead of including it in the application package. This is for various good reasons (maintenance, security, package size, ...).
Yes, I know; it's why I changed the 'ledgersmb' pkg to use the Debian dojo pkgs when LSMB started using dojo 3 years ago. I also know that the policy [1] is a 'should' not a 'must'. OTOH; I don't yet know how much, if any, the version difference might actually make. Especially over a patch level difference like that between 1.14.2 & 1.14.3. Which likely means that I should really focus first on gettting the 'ledgersmb' pkg working with autopkgtest.
The difference in lines of code between 1.14.0 and 1.14.1 was pretty small, but it did mean broken file-uploads in LedgerSMB on Safari. Yes, getting the package build working with automated testing seems like a great way to do the required quality assurance needed to make sure another Dojo version can be used with LedgerSMB than the one that comes with the tar archive. -- Bye, Erik. http://efficito.com -- Hosted accounting and ERP. Robust and Flexible. No vendor lock-in.