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

[Jose M Rodriguez]

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