-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add snappy #193
Add snappy #193
Changes from 1 commit
fa75e8f
c2c9d65
f966259
0a08730
8cfcabb
1833a82
67d524b
7823022
0d2f62d
abdb9b7
d8f3b10
b9df726
147a46b
4175c0f
55395fc
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,12 @@ | ||
cd %SRC_DIR% | ||
git clone https://github.com/kmanley/snappy-msvc | ||
cd snappy-msvc | ||
|
||
set SLN_FILE=snappy.sln | ||
set SLN_CFG=Release | ||
if "%ARCH%"=="32" (set SLN_PLAT=Win32) else (set SLN_PLAT=x64) | ||
|
||
REM Build step | ||
devenv "%SLN_FILE%" /Build "%SLN_CFG%|%SLN_PLAT%" | ||
if errorlevel 1 exit 1 | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Drop extra newline. |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
#!/bin/env bash | ||
|
||
./autogen.sh | ||
./configure --prefix=$PREFIX | ||
make | ||
make install | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. make check? |
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
{% set version = "1.1.3" %} | ||
|
||
package: | ||
name: snappy | ||
version: {{ version }} | ||
|
||
source: | ||
git_url: https://github.com/google/snappy.git | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could we just download the tarball instead? Should be faster and may avoid a few extra configuration steps. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are tarballs preferred over repositories? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. IMHO, yes, for many reasons:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also, they may have run |
||
git_tag: {{ version }} | ||
|
||
about: | ||
home: https://github.com/google/snappy | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Needs a summary line. |
||
license: Other | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Looks like BSD 3-Clause. Could we put that here? I know the data is licensed differently, but don't think we want to distribute that anyways. |
||
license_file: COPYING | ||
license_family: Other | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Needs maintainer. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Again think this is BSD. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this is not part of the source code.
Can we clone a specific hash? (Although it seems that nothing is updated in almost 4 years!)
PS: Maybe we should just download those files and ship with the recipe. Just adding a comment about that repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point.
Like this. Maybe with a shallow clone too?
Not sure how I feel about this.
So, I'm almost leaning to having
snappy-msvc
broken out as its own package that just does the git clone with a sha. It feels cleaner IMHO. Maybe others have thoughts about this? @pelson?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this case, the two important files are the
sln
andvcproj
.stdint
can be got from themsinttypes
package (https://github.com/conda-forge/msinttypes-feedstock). @groutr - are you happy to just copy those files into the recipe?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm fine with copying those into the recipe.