veriT: 201410 -> 201506 #9451
veriT: 201410 -> 201506 #9451
Conversation
|
|
||
| enableParallelBuilding = false; |
There was a problem hiding this comment.
Can you keep this line please? If it was there it was to point parallel building is not suitable for this package. Removing it makes people think it could be enabled.
There was a problem hiding this comment.
I was the one who added the line in the first place. Since then, upstream has switched to a new autotools-based build system which seems to work just fine with parallel builds (I ran nix-build --check a few times and it didn't complain).
There was a problem hiding this comment.
Parallel builds are disabled by default, you have to enableParallelBuilding = true.
There was a problem hiding this comment.
Sorry for being unclear; I locally added enableParallelBuilding = true to the derivation, built it several times with nix-build --check and verified with top that is indeed being built in parallel.
There was a problem hiding this comment.
Ok so you didn't enable it in this PR.
|
Built locally thanks. |
No description provided.