1019246 - Shut up stupid NSS build spam

Status: RESOLVED FIXED

(NSS :: Build, defect)

Description

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Attachment: Quiet NSS build

I am incredibly sick of seeing:

There are no private exports.
There are no private exports.
There are no private exports.
cd params; c:/Users/vladimir/bin/mozmake private_export
cd params; c:/Users/vladimir/bin/mozmake private_export
cd params; c:/Users/vladimir/bin/mozmake export
There are no private exports.
There are no private exports.
There are no private exports.
cd results; c:/Users/vladimir/bin/mozmake private_export
cd results; c:/Users/vladimir/bin/mozmake private_export
cd results; c:/Users/vladimir/bin/mozmake export
There are no private exports.
cd store; c:/Users/vladimir/bin/mozmake private_export
There are no private exports.
There are no private exports.
cd store; c:/Users/vladimir/bin/mozmake private_export
cd store; c:/Users/vladimir/bin/mozmake export
There are no private exports.
cd top; c:/Users/vladimir/bin/mozmake private_export
There are no private exports.
There are no private exports.
cd top; c:/Users/vladimir/bin/mozmake private_export
cd top; c:/Users/vladimir/bin/mozmake export
There are no private exports.
cd util; c:/Users/vladimir/bin/mozmake private_export
There are no private exports.
There are no private exports.
There are no private exports.
cd util; c:/Users/vladimir/bin/mozmake export
cd util; c:/Users/vladimir/bin/mozmake private_export
etc.

This just needs to die.

Comment 1

[Mike Hommey [:glandium]]

Comment on attachment 8432823 [details] [diff] [review]
Quiet NSS build

Review of attachment 8432823 [details] [diff] [review]:
-----------------------------------------------------------------

NSS is its own module, with different peers than the build system.

::: security/nss/coreconf/rules.mk
@@ +771,4 @@
>  	$(INSTALL) -m 444 $^ $(PRIVATE_EXPORT_DIR)
>  else
>  private_export:: 
> +	$(NULL)

That shouldn't be needed.

@@ +771,5 @@
>  	$(INSTALL) -m 444 $^ $(PRIVATE_EXPORT_DIR)
>  else
>  private_export:: 
> +	$(NULL)
> +#	@echo There are no private exports.;

If you want to remove something, don't comment it, remote it ;)

Comment 2

[Brian Smith (:briansmith, :bsmith, use NEEDINFO?)]

Comment on attachment 8432823 [details] [diff] [review]
Quiet NSS build

Review of attachment 8432823 [details] [diff] [review]:
-----------------------------------------------------------------

This annoying output is an indicator of inefficiencies in the NSS build process. If we remove the output then those inefficiencies will be (further) hidden. This part, in particular, is really slow on Windows. A better way of getting rid of this annoying output would be to centralize all the exports into a single, efficient, Makefile rule.

That said, r- for the reasons Mike noted, but I am happy to r+ the removal of these print statements, unless anybody else objects.

::: security/nss/coreconf/rules.mk
@@ +771,5 @@
>  	$(INSTALL) -m 444 $^ $(PRIVATE_EXPORT_DIR)
>  else
>  private_export:: 
> +	$(NULL)
> +#	@echo There are no private exports.;

I agree with Mike's comments.

::: security/nss/coreconf/ruleset.mk
@@ +204,4 @@
>  
>  ifdef DIRS
>  define SUBMAKE
> +#+@echo "cd $2; $(MAKE) $1"

Ditto here; don't comment it out; remove it.

Comment 3

[Tim Taubert [:ttaubert] (inactive)]

NSS is built with GYP these days.