Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

include icl headers #27

Merged
merged 1 commit into from
May 4, 2016
Merged

Conversation

jayhesselberth
Copy link
Contributor

This PR incorporates boost headers for the ICL (Interval Container Library).

I used the CreateBoost.sh script to incorporate the new headers, and below is a diff of that script.

index cd682d0..f4f2252 100755
--- a/local/scripts/CreateBoost.sh
+++ b/local/scripts/CreateBoost.sh
@@ -13,7 +13,7 @@ pkgdir="${HOME}/git/bh"
 boosttargz="boost_1_60_0.tar.gz"
 ## -- current package version and date (and other metadata as needed)
 version="1.60.0-0"
-date="2016-2-20"
+date="2016-5-4"



@@ -121,7 +121,7 @@ bcp --boost=${boostroot}  ${boostlibs}  ${pkgincl}   > /dev/null  2>&1
 # Plus phoenix (cf [github] issue ticket #19)
 boostextras="filesystem spirit foreach algorithm iostreams \
             dynamic_bitset heap any circular_buffer geometry fusion graph \
-            multiprcecision phoenix bimap"
+            multiprcecision phoenix bimap icl"

 bcp --boost=${boostroot}  ${boostextras}   ${pkgincl}   > /dev/null   2>&1

For reference, this is a "middleweight" addition to the package:

$ du -h /usr/local/include/boost/icl/
232K    /usr/local/include/boost/icl/concept
144K    /usr/local/include/boost/icl/detail
16K /usr/local/include/boost/icl/predicates
208K    /usr/local/include/boost/icl/type_traits
892K    /usr/local/include/boost/icl/

@eddelbuettel
Copy link
Owner

Thanks a lot. And sorry for the massive repo size. I mistakenly commit the boost sources a few times earlier in the project -- bad idea.

What you did with the diff is the right idea. I'll see about merging and then possibly catching up to one or two more change requests that were pending.

@eddelbuettel
Copy link
Owner

(And bcp may make it more middleweight than the df -h over its own directory. Then again it may pull in other depends but as we converge towards more and more Boost than side is less likely....)

@eddelbuettel eddelbuettel merged commit a31220c into eddelbuettel:master May 4, 2016
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.

None yet

2 participants