ChiliProject is not maintained anymore. Please be advised that there will be no more updates.

We do not recommend that you setup new ChiliProject instances and we urge all existing users to migrate their data to a maintained system, e.g. Redmine. We will provide a migration script later. In the meantime, you can use the instructions by Christian Daehn.

Remove included lib/faster_csv.rb (Bug #517)


Added by Eric Davis at 2011-07-11 08:30 pm. Updated at 2011-08-21 06:54 pm.


Status:Closed Start date:2011-07-11
Priority:Normal Due date:
Assignee:- % Done:

0%

Category:Libraries
Target version:2.2.0
Remote issue URL: Affected version:2.0.0

Description

Should use the gem (1.8.x) or the built in CSV engine (1.9). Bundling it in lib/

  • prevents us from upgrading it easily
  • bloats the app size (download and disk, not running memory size)

Associated revisions

Revision 0b57e936
Added by Felix Schäfer at 2011-08-21 07:26 pm

Rip faster_csv out of lib into the Gemfile. #517

Revision 57ecf525
Added by Felix Schäfer at 2011-08-21 07:58 pm

Merge pull request #100 from thegcat/feature/517-put_faster_csv_in_the_gemfile

Rip faster_csv out of lib into the Gemfile. #517

Revision 9dae76d5
Added by Felix Schäfer at 2011-08-21 11:36 pm

This should fix it once and for all… #517

(Sorry for the commit-noise)

History

Updated by Felix Schäfer at 2011-07-11 09:42 pm

Would work for me.

Updated by Felix Schäfer at 2011-08-21 05:29 pm

Yeah, pull request 100

Scheduling this for 2.2.0, not sure how additions to the Gemfile fly with semver though, so if anyone thinks it shouldn't go into a minor, please reschedule it for 3.0.

  • Target version set to 2.2.0
  • Status changed from Open to Ready for review

Updated by Felix Schäfer at 2011-08-21 05:59 pm

Had a short chat with Holger on IRC, he agreed we could do it in a minor release, merged in 57ecf52.

Just to make sure, I'll see if our install docs have bundler instructions for minor updates.

  • Status changed from Ready for review to Closed

Updated by Felix Schäfer at 2011-08-21 06:54 pm

Felix Schäfer wrote:

Just to make sure, I'll see if our install docs have bundler instructions for minor updates.

(the docs tell to bundle install after an update, so everything is good)

Also available in: Atom PDF