Skip to content

Commit

Permalink
wt: fixup build by using older boost again
Browse files Browse the repository at this point in the history
160 seems a popular version in nixpkgs.
  • Loading branch information
vcunat committed Oct 20, 2017
1 parent 52350b5 commit 9f24090
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion pkgs/top-level/all-packages.nix
Expand Up @@ -10967,7 +10967,9 @@ with pkgs;

wiredtiger = callPackage ../development/libraries/wiredtiger { };

wt = callPackage ../development/libraries/wt { };
wt = callPackage ../development/libraries/wt {
boost = boost160; # 165: error: invalid conversion from 'const char*'...
};

wxGTK = wxGTK28;

Expand Down

5 comments on commit 9f24090

@vcunat
Copy link
Member Author

@vcunat vcunat commented on 9f24090 Oct 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/cc maintainer @juliendehos.

@juliendehos
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On my machine, wt seems to compile perfectly using boost 1.65. Boost 1.60 is also ok for me.

@vcunat
Copy link
Member Author

@vcunat vcunat commented on 9f24090 Oct 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, strange. The logs on Hydra: https://hydra.nixos.org/build/62686893

@juliendehos
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah ok, this is wt-3.3.6. But the current version in master is wt-4.0.0.

@vcunat
Copy link
Member Author

@vcunat vcunat commented on 9f24090 Oct 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, right, it was updated on master because of boost but those commits weren't yet on staging where the boost update happened. Reverted in 90435ba and tidied in c3263fc.

Please sign in to comment.