ports/75580: [PATCH] www/firefox: main merge from debian

[Jose M Rodriguez]

The following reply was made to PR ports/75580; it has been noted by GNATS.

From: Jose M Rodriguez <josemi at freebsd.jazztel.es>
To: "Jose M Rodriguez (freebsd)" <josemi at freebsd.jazztel.es>
Cc: FreeBSD-gnats-submit at FreeBSD.org, gnome at FreeBSD.org
Subject: Re: ports/75580: [PATCH] www/firefox: main merge from debian
Date: Wed, 29 Dec 2004 12:22:53 +0100

 Jose M Rodriguez (freebsd) wrote:
 
 Well, I'll do just a more detailed note and  you may close this.
 I'll send-pr another path with what I think  is  a must and it isn't in 
 the tree
 <snip/>
 
 >n this patchset:
 >
 >files/patch-uriloader_exthandler_unix_nsGNOMERegistry.cpp
 >  - reprocopy fixes from mozilla (same code and use).
 >  
 >
 This must go ASAP.  make external URIs work on firefox/thundebird and gnome
 The patch is from joe@ and must go to both firefox/thunderbird.
 
 >files/patch-browser_app_mozilla.in
 >  - fix for non posix expr (RELENG_4_9?)
 >  - protect against missleave as test options
 >  
 >
 I'm working on a new script ( see ports/75595 ).  My plans is go to 
 FILESDIR now and try merge upstream after test. thinking about move part 
 of this to MOZ_PIS and merge the rest.
 
 >files/patch-browser_app_Makefile.in
 >  - small fix from debian
 >  
 >
 $(destdir) -> $(DESTDIR).  May we need this?
 
 >files/patch-browser_app_profile_extensions_classic_install.rdf.in
 >  - small fix from debian
 >
 >  
 >
 See patch.
 
 >files/patch-browser_app_profile_firefox.js
 >  - disable default browser check
 >  
 >
 Dont work, at last, for me.  See that tools like gnome-control-center 
 are the right way on Unix.
 
 >  - disable app update, from debian
 >  - disable autoUpdate, from debian
 >  
 >
 This seems OK.  We must use tools like portupgrade for this.
 
 >  - Use LANG environment variable to choose locale, from debian
 >  
 >
 We may live without this and the rest of chrome://global
 
 >  - enable negotiate extension
 >
 >  
 >
 We may live without this, but seems OK
 
 >files/patch-browser_base_content_aboutDialog.css
 >  - use MACOSX defaults, from debian
 >
 >  
 >
 same
 
 >files/patch-browser_components_prefwindow_content_pref-advanced.xul
 >  - disable app autoUpdate in XUL, from debian
 >
 >  
 >
 seems OK.  Use portupgrade for this.
 
 >files/patch-content_events_src_nsEventStateManager.cpp
 >  - mouse extended events, from debian (please, review)
 >
 >  
 >
 We can live without this. I don't have multi-wheel mouse.
 
 >files/patch-gfx_src_gtk_fontEncoding.properties
 >  - correct ttf symbol font encodings
 >
 >  
 >
 Seems OK use adobe-fontmetrics for Unix symbols
 
 >files/patch-htmlparser_src_nsParser.cpp
 >  - from debian (please, review)
 >
 >  
 >
 We can live without this.
 
 >files/patch-layout_html_document_src_html.css
 >  - from debian (please, review)
 >  
 >
 I can't get an opinion about this.
 
 >files/patch-modules_libpref_src_init_all.js
 >  - chrome://global, from debian
 >    (better import using reimplace in the works)
 >  - mouse extended events, from debian
 >
 >  
 >
 We can live without this.
 
 >files/patch-netwerk_protocol_ftp_src_nsFtpConnectionThread.cpp
 >  - ftp conn fixes, from debian (please test)
 >
 >  
 >
 I doesn't have problems with ftp.
 
 >files/patch-toolkit_mozapps_extensions_content_extensions.js
 >  - new debian extension manager (please, test)
 >  (initial import, before full support of extensions/themes from ports)
 >  
 >
 This maybe ok, but review
 
 >files/patch-toolkit_mozapps_extensions_src_nsExtensionManager.js.in
 >  - new debian extension manager (please, test)
 >  (initial import, before full support of extensions/themes from ports)
 >  
 >
 No comments about this
 
 >files/patch-toolkit_mozapps_update_src_nsUpdateService.js.in
 >  - disable app updates, from debian
 >  
 >
 This seems ok, use portupgrade for this.
 
 >files/patch-widget_public_nsGUIEvent.h
 >  - extended mouse events, from debian
 >
 >files/patch-widget_src_gtk2_nsWindow.cpp
 >  - extended mouse events, from debian
 >  
 >
 We can live without this (multi-wheel mouse)
 
 >files/patch-xpcom_io_nsAppDirectoryServiceDefs.h
 >  - support for user search dir, from debian
 >    doubt if of interest, but seems don't hurt
 >
 >files/patch-xpcom_io_nsAppFileLocationProvider.cpp
 >  - support for user search dir, from debian
 >    doubt if of interest, but seems don't hurt
 >
 >files/patch-xpfe_components_search_src_nsInternetSearchService.cpp
 >  - support for user search dir, from debian
 >    doubt if of interest, but seems don't hurt
 >
 >  
 >
 We can live without this
 
 >files/patch-xpinstall_packager_unix_browser.jst
 >  - chrome://global, from debian
 >
 >  
 >
 We can live without chrome://global
 
 >files/firefox.desktop.in
 >  - i18n
 >
 >  
 >
 Please, disable StartupNotification.  Problems with mozilla-xremote.
 
 <snip/>
 
 --
   josemi