599283 - --disable-storage breaks the builds

Status: RESOLVED WONTFIX

(Core :: General, defect)

Description

[Mounir Lamouri (:mounir)]

Output:
nsNetModule.cpp
In file included from /home/volkmar/projects/mozilla/mc-asserts/netwerk/build/nsNetModule.cpp:63:
/home/volkmar/projects/mozilla/mc-asserts/netwerk/build/../cache/nsDiskCacheDeviceSQL.h:46:35: error: mozIStorageConnection.h: No such file or directory
[...]
/home/volkmar/projects/mo/home/volkmar/projects/mozilla/mc-asserts/netwerk/build/../cache/nsDiskCacheDeviceSQL.h:46:35: error: mozIStorageConnection.h: No such file or directoryzilla/mc-asserts/netwerk/build/../cache/nsDiskCacheDeviceSQL.h:267: error: template argument 1 is invalid
/home/volkmar/projects/mozilla/mc-asserts/netwerk/build/../cache/nsDiskCacheDeviceSQL.h:268: error: ‘mozIStorageStatement’ was not declared in this scope
/home/volkmar/projects/mozilla/mc-asserts/netwerk/build/../cache/nsDiskCacheDeviceSQL.h:268: error: template argument 1 is invalid
make[5]: *** [nsNetModule.o] Error 1
make[4]: *** [libs] Error 2
make[3]: *** [libs_tier_platform] Error 2
make[2]: *** [tier_platform] Error 2
make[1]: *** [default] Error 2
make: *** [build] Error 2

I didn't look at the error (quite late) so it might be trivial.

Comment 1

[Mounir Lamouri (:mounir)]

This error isn't really trivial and I'm wondering if it really make sense to be able to disable storage now?

Comment 2

[(not currently active) Ted Mielczarek]

I would guess that it's not a valid option for Firefox builds, anyway.

Comment 3

[Sylvestre Ledru [:Sylvestre]]

I don't think we need it anymore (or it is fixed)