Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=596138 --- Comment #6 from Kai Engert (kaie) <kengert@xxxxxxxxxx> 2010-05-27 04:27:56 EDT --- (In reply to comment #4) > > Some non-packaging comments: > - Having a configuration tool that will require rebuilds at every xulrunner > rebase is not nice. In 99% of xulrunner rebases it will NOT require a rebuild :) There is no binary dependency between xulrunner and nss-gui, the integration works via dynamic runtime hooks. nss-gui asks to bring up dialogs by their internal URL-like addresses, and uses xml-like instructions to modify the contents of the UI. Only on major version upgrades we'll need to test whether the UI hooks need to be adjusted, and the required changes are going to be small. > - This tool is *extremely* jargon-heavy. I'm not sure how you'd explain it to > someone not familiar with all the concepts already. Agreed, but do you think this is different from the UI shown at the respective level of Firefox preferences? I guess we don't need to make this tool a default install, so only people who need it will use it. > - I don't see any real obvious use for the 'error console' and 'about:config' > dialogs. Certainly 'about:config' seems superfluous; is there a reason that I > need to be able to change nss-gui's cookie preferences? Yeah, these are not really necessary. I used them during development and testing. I could use a xulrunner-level preference to switch those debug helpers on and off, make them off by default, offer a --debug switch for the nss-gui startup wrapper script, and have the wrapper script modify the pref before nss-gui xulrunner startup. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review