Skip to content

{tools}[dummy/] Lucene-Geo-Gazetteer v20170718#5064

Merged
verdurin merged 3 commits intoeasybuilders:developfrom
boegel:20170825131739_new_pr_Lucene-Geo-Gazetteer20170718
Aug 25, 2017
Merged

{tools}[dummy/] Lucene-Geo-Gazetteer v20170718#5064
verdurin merged 3 commits intoeasybuilders:developfrom
boegel:20170825131739_new_pr_Lucene-Geo-Gazetteer20170718

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Aug 25, 2017

(created using eb --new-pr)

@boegel boegel added this to the 3.4.0 milestone Aug 25, 2017
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 25, 2017

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2415.golett.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/e1f6a7aae91066adb494108564e253a8 for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 25, 2017

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2047.delcatty.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/34411f37645a0c11a27f16717fb4c8e2 for a full test report.


homepage = 'https://github.com/chrismattmann/lucene-geo-gazetteer'
description = """A command line gazetteer built around the Geonames.org dataset, that uses the Apache Lucene library
to create a searchable gazetter."""
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@boegel Please use the same spelling of gazetteer within the same sentence...

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Copy-paste from the README at https://github.com/chrismattmann/lucene-geo-gazetteer, but I guess it's a typo... I'll fix it, also sent them a PR: chrismattmann/lucene-geo-gazetteer#26

checksums = ['2ed355bfdcf865d815df9dc52ee747eeb8a15d065ef159ebf933ad36e1555ad4']

builddependencies = [('Maven', '3.5.0')]
dependencies = [('Java', '')]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@boegel This is odd - does it really only need any version of Java?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Hah! This is obviously wrong, it should specify a particular version of Java, will fix.

@easybuilders easybuilders deleted a comment from boegelbot Aug 25, 2017
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 25, 2017

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2415.golett.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/6cad9c02e2626d75aa43a0bfb137dece for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Aug 25, 2017

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2047.delcatty.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/2f9a6b8e5fde912f7504a1783cabc907 for a full test report.

@easybuilders easybuilders deleted a comment from boegelbot Aug 25, 2017
@verdurin
Copy link
Copy Markdown
Member

Test report by @verdurin
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
ca003.camp.thecrick.org - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2640 v3 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/2c990dd90ccf16293d4fa3f99e7218e6 for a full test report.

Copy link
Copy Markdown
Member

@verdurin verdurin left a comment

Choose a reason for hiding this comment

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

Looks fine.

@verdurin
Copy link
Copy Markdown
Member

Going in, thanks @boegel!

@verdurin verdurin merged commit 174c48e into easybuilders:develop Aug 25, 2017
@boegel boegel deleted the 20170825131739_new_pr_Lucene-Geo-Gazetteer20170718 branch June 22, 2020 08:01
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.

2 participants