308580 - some configure.in cleanup

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Doug Turner (:dougt)]

Comment 1

[Doug Turner (:dougt)]

Attachment: patch v.1

1) Not all MOZ_XUL_APPS need or want updater.  This patch fixes this.
2) Disables View Source in minimo.
3) Removes data: protocol from minimo.
4) Removes XFT from minimo.

Comment 2

[Doug Turner (:dougt)]

Comment on attachment 196096 [details] [diff] [review]
patch v.1

there is an extra '=' char in the patch:

-  NECKO_PROTOCOLS="about,data,http,file,res,jar"
+  NECKO_PROTOCOLS_DEFAULT=="about,http,file,res,jar"

i have removed it.

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 196096 [details] [diff] [review]
patch v.1

>-  NECKO_PROTOCOLS="about,data,http,file,res,jar"
>+  NECKO_PROTOCOLS_DEFAULT=="about,http,file,res,jar"

I think you should keep data.  It's small and useful, and we may want to
encourage its use on mobile devices.

And you certainly want == rather than =.

Other than that this looks fine.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Er, = rather than ==.

Comment 5

[Doug Turner (:dougt)]

Checking in configure.in;
/cvsroot/mozilla/configure.in,v  <--  configure.in
new revision: 1.1528; previous revision: 1.1527
done


I didn't change the data: per dbaron's request.  Also, i am using space
delimiters in the default protcols like how it is today.