Skip to content
This repository

Make runtime dependency on some gems optional #24

Closed
thbar opened this Issue June 14, 2011 · 1 comment

1 participant

Thibaut Barrère
Thibaut Barrère
Owner
thbar commented June 14, 2011

Some destinations or processors use a number of gems, including tmail, net-sftp, zip and spreadsheet.

Currently, the requires occur unchecked, which means the etl command will raise an error if any of these is not installed.

Thibaut Barrère thbar referenced this issue from a commit June 14, 2011
Thibaut Barrère Currently the end-user has to reference all the third-party gems used…
… by components of aw-etl. This temporarily solves #24 until we disable loading these components when their dependencies are not installed.
c328db4
Thibaut Barrère
Owner
thbar commented June 14, 2011

Temporary fix in c328db4: automatically install the dependencies when installing activewarehouse-etl.

The proper fix is I think to check if require fails and avoid defining the dependent transforms.

Thibaut Barrère thbar closed this issue from a commit June 20, 2011
Thibaut Barrère Use optional_require for spreadsheet, nokogiri, tmail, net/imap, net/…
…pop. Fix #24: Make runtime dependency on some gems optional.
48ebd02
Thibaut Barrère thbar closed this in 48ebd02 June 19, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.