Skip to content

pktgen: build with the same CFLAGS as dpdk#15962

Merged
domenkozar merged 1 commit intoNixOS:masterfrom
abuibrahim:master
Jun 8, 2016
Merged

pktgen: build with the same CFLAGS as dpdk#15962
domenkozar merged 1 commit intoNixOS:masterfrom
abuibrahim:master

Conversation

@abuibrahim
Copy link
Contributor

@abuibrahim abuibrahim commented Jun 3, 2016

Motivation for this change

Unbreak the master build failure https://hydra.nixos.org/build/36613719
pktgen requires SSE3 instructions - same as DPDK.

Interestingly build wasn't failing on release-16.03 with gcc-4.9.3 but fails with gcc-5.3.0.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • OS X
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@joachifm
Copy link
Contributor

joachifm commented Jun 7, 2016

cc @domenkozar

@abuibrahim abuibrahim closed this Jun 8, 2016
@abuibrahim
Copy link
Contributor Author

@domenkozar, this was a separate PR on master, which haven't been merged yet.

@joachifm
Copy link
Contributor

joachifm commented Jun 8, 2016

Why close it?

@abuibrahim
Copy link
Contributor Author

Wait, did I close it? My bad. I did another rebase and force pushed.

@abuibrahim abuibrahim reopened this Jun 8, 2016
@domenkozar domenkozar merged commit 964665e into NixOS:master Jun 8, 2016
@abuibrahim abuibrahim mentioned this pull request Jun 10, 2016
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants