Bug 249947 - databases/arrow: patch including expanded options
Summary: databases/arrow: patch including expanded options
Status: In Progress
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Yuri Victorovich
URL:
Keywords: patch, patch-ready
Depends on:
Blocks:
 
Reported: 2020-09-27 19:07 UTC by Matías Pizarro
Modified: 2020-10-20 19:33 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (yuri)


Attachments
Patch expanding available Arrow options (17.14 KB, patch)
2020-09-27 19:07 UTC, Matías Pizarro
no flags Details | Diff
would this be a cleaner patch? (16.79 KB, patch)
2020-09-27 19:12 UTC, Matías Pizarro
no flags Details | Diff
Include python option (25.87 KB, patch)
2020-09-28 17:54 UTC, Matías Pizarro
no flags Details | Diff
Latest full patch (30.16 KB, patch)
2020-09-29 15:11 UTC, Matías Pizarro
no flags Details | Diff
Update to Arrow v2.0.0 (32.51 KB, patch)
2020-10-20 19:33 UTC, Matías Pizarro
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matías Pizarro 2020-09-27 19:07:01 UTC
Created attachment 218367 [details]
Patch expanding available Arrow options

Hey Yuri,

Thanks for getting things going with Arrow.
Here's a patch expanding the Arrow port build options:

COMPUTE
JSON
CSV
FILESYSTEM
PARQUET
FLIGHT
DATASET
HDFS
GANDIVA
BUILD_UTILITIES

I also have a py-arrow port but I still need to work out some wrinkles around the flavours.


Best,
Comment 1 Matías Pizarro 2020-09-27 19:12:55 UTC
Created attachment 218368 [details]
would this be a cleaner patch?

Port diff generated a few extraneous things, would this be a better patch?
Comment 2 Matías Pizarro 2020-09-28 17:54:08 UTC
Created attachment 218379 [details]
Include python option
Comment 3 Yuri Victorovich freebsd_committer 2020-09-28 18:29:20 UTC
Hi Matias,

Thank you for the patch.

Could you please fix these 2 items:
1. Sort pkg-plist according to file names. Add %%{OPTION}%% tags to the untagged plist file, not add option-based sections to the bottom of plist. When a new version would arrive, comparing plists would be easier.
2. USES=ssl isn't required when all new options are "off". Could you please move USES=ssl to options that need it?

Thank you,
Yuri
Comment 4 Matías Pizarro 2020-09-29 15:11:50 UTC
Created attachment 218407 [details]
Latest full patch
Comment 5 Matías Pizarro 2020-09-29 21:04:43 UTC
There you go Yuri.

1.pkg-plist is sorted according to file name. Is that the way you wanted it?

2. USE_OPTION=ssl has been added to the relevant options and removed from the general scope

3. I've added two more options: build TESTS and S3 support

Thanks for pointing me in the right direction, keep it coming!

Best,

Matías
Comment 6 Yuri Victorovich freebsd_committer 2020-09-29 21:44:10 UTC
(In reply to Matías Pizarro from comment #5)

Yes, thank you, Matías.

I will do some extra testing and commit within a few days.

Best,
Yuri
Comment 7 Matías Pizarro 2020-10-20 19:33:00 UTC
Created attachment 218930 [details]
Update to Arrow v2.0.0

Hi Yuri,

I have updated the port to handle Arrow v2.0.0 release.
Can I help you with your tests in any way?

Best,

Matías