Closed Eldara98 closed 9 years ago
@Eldara98 How goes the testing? This needs to be done ASAP...
Oops I got heredocs wrong... Using << 'EOF'
will suppress parameter substitution, which we specifically don't want, and won't suppress leading tabs, so client-config is broken with this PR. The syntax needed is <<- EOF
. Quoting the eof string suppresses parameter substitution, using <<-
suppresses leading tabs.
@Eldara98 Final call: is this working as expected in all areas?
Yes, although one final test is necessary to check profile uploads to the server.
@Eldara98 Good point. Can you do that tomorrow?
@Xenopathic Yes, I see no reason why not.
Alongside moving as much Firefox configuration out of
pre-session
, this now uploads the entire Firefox directory to allow for preference permanance. The size of the uploaded directory has a balance to hopefully ensure it is not too large by locking extension installation with thexpinstall.enabled
preference locked tofalse
.cc: @Xenopathic