By Developers, For Developers
| PDF Pg | Paper Pg | Type | Description | Fixed on | Comments |
|---|---|---|---|---|---|
| x | SUGGEST | Suggest telling readers to install PostgreSQL before running initial ‘bundle install’ command. The pg gem will fail otherwise. On Ubuntu 10.10 (I know I’m behind) needed to run ‘sudo apt-get install postgresql postgresql-server-dev-8.4’ before the pg gem would install. Both packages are needed. | 2012-03-14 | ||
| 8 | ERROR | ‘bundle install’ should be re-run after the jdbc adapters are added to the gemfile, and before ‘rake db:migrate’. | 2012-03-08 | ||
| 8 | ERROR | ‘rake db:migrate’ fails due to missing JS runtime. Suggest adding therubyrhino gem to gem file. | 2012-03-26 | ||
| 106 | TYPO | Superfluous ‘w’: This added two dependencies t_w_o our Gemfile | 2012-03-11 | ||
| 11 | ERROR | “$ RAILS_ENV=test rake db:migrate” fails with “rake aborted! can’t convert nil into String”. Not sure why. Here’s the stack trace if it helps. ~/code/deployingJruby/twitalytics$ RAILS_ENV=test rake db:migrate —trace | 2012-03-27 | ||
| 22 | TYPO | Should be “We’ll” - “Ee’ll be using it to provision our virtual machine with… ” | 2012-03-11 | ||
| ix | TYPO | “We can check that JRuby was install correctly…” should be “installed” | 2012-03-14 | ||
| 7 | SUGGEST | I have started the twitalytics exercise using Ruby 1.9.3 (the MRI) under RVM (Ruby Version Manager). I have created a gem set 3.2.1 for twitalytics. Installed bundler and used bundle install to get all the dependencies installed. Then I installed and used warbler - no problem. Then I installed the jruby-lint as well but the jrlint command was returning an error: “JRuby-Lint must be run under JRuby”. | 2012-03-27 | ||
| 10 | TYPO | Inconsistency in Gemfile in terms of using single or double quotes. In the Gemfile you have single quotes, like: gem ‘jquery-rails’ where in the book you are putting: gem “get_back” It would be an elegant practice to keep it one way. | 2012-03-28 | ||
| 46 | TYPO | There is mention setting JRUBY_HOME even though it’s not being set. | 2012-03-26 | ||
| 62 | TYPO | Should be 0/15 not 0/5 “For example, 0/5 in the seconds field means the seconds 0, 15, 30, and 45” | 2012-03-28 | ||
| 48 | SUGGEST | Suggest committing changes after successful deploy to tomcat, and before pushing to cloudbees, and then again after cloudbees. As it is now, there are no commits in chapter 3. This may cause some readers to be confused when they run “git checkout -b trinidad jruby” in chapter 4 with uncommitted files. | 2012-04-02 | ||
| 58 | SUGGEST | Although it is implied, you may want to suggest stopping trinidad before starting it again. You start trinidad after each extension, but never remind the reader to shut it down. | 2012-04-02 | ||
| 65 | TYPO | Footnote 12 is plain text, instead of a working link. | 2012-04-02 | ||
| 44 | SUGGEST | Should tell people what the different Tomcat logs are under /var/lib/tomcat6/logs | 2012-04-02 | ||
| 54 | TYPO | This sentence doesn’t quite make sense - “…you’ll learn about two JRuby servers these kind of tasks.” Maybe “…servers that support these kind of tasks.” | 2012-03-31 | ||
| 36 | SUGGEST | After making the changes to add the warbler configuration file, the book has us just add that config/warble.rb file to the git repo. However, we also modified the environments/production.rb file as well. Are you wanting the user to commit that change to git as well? | 2012-04-02 | ||
| 127 | TYPO | Missing word “to”. - “We want [to] use an initializer file to load it conditionally,…” | 2012-03-31 | ||
| 49 | TYPO | “But Rails looks for our assets under the /assets path be default.” should read “But Rails looks for our assets under the /assets path by default.” | 2012-03-31 | ||
| 0 | SUGGEST | I deploy around 10-20 times a day. Doing this requires zero downtime deploys. I do this right now with unicorn and signals. I’m looking to move to Torquebox, but I’d need the ability to do seamless deploys. Searching the PDF for “downtime” only had one mention about Kirk/Jetty. It would be fantastic to show how torquebox can support zero downtime deploys (hopefully both rails and the services). | 2012-04-18 | ||
| xviii | TYPO | link 17 should be pointing to the Trinidad wiki, not the google group. | 2012-04-17 | ||
| 148 | TYPO | remove one superfluous “the”: Before we use the the deployer… | 2012-04-17 | ||
| 155 | TYPO | missing ‘to’: In order to demonstrate | 2012-04-17 | ||
| 22 | TYPO | “MRI cannot handle multiple request in the same” should be “MRI cannot handle multiple requests in the same” | 2012-05-01 | ||
| 26 | TYPO | “This is where we’ll find all of HTML files” should be “This is where we’ll find all of the HTML files” | 2012-05-01 | ||
| 27 | TYPO | “but Twitalytics is is more complex” | 2012-05-01 | ||
| 38 | TYPO | Either “Before we move on, let’s commit our changes to the warbler branch with the git commit command.” or “Now let’s commit our changes to the Git repository”, not both. | 2012-05-01 | ||
| 46 | TYPO | “so we’ll included in our base environment” should be “so we’ll include it in our base environment” | 2012-05-01 | ||
| 47 | TYPO | “We’ll be adding to this class, but for now we’ve defined a variable to it that contains the location where we want to install our JRuby runtime” should (maybe) be “but for now we’ve just defined a variable that contains”. | 2012-05-01 | ||
| 113 | TYPO | Sixth line down reads: Missing “be”. | 2012-05-27 | ||
| 26 | TYPO | “The simplest way to get JRuby onto our virtual machine is is by using the system’s package manager, just as we did for the JVM.” Remove the extra ‘is’. | 2012-05-27 | ||
| 32 | TYPO | “You can use any value you prefer for the password, but remember it because we’ll needed it when we run our application’s database settings.” Should be “…we’ll need it…” | 2012-05-27 | ||
| 79 | ERROR | Section 5.2 is missing File[$trinidad_home], which is the code. | |||
| 73 | 53 | ERROR | bees deploy twitalytics.war -appid yourAccount/twitalytics | 2013-03-28 | |
| 100 | 81 | TYPO | In the following code, shouldn’t owner => vagrant be owner => “vagrant”? (vagrant in quotes) It seems to work either way, but the inconsistency is confusing to someone learning and paying attention to detail. | ||
| 51 | ERROR | curl command should have should include dest filename | 2013-03-28 | ||
| 0 | SUGGEST | Add Gem versions to Gemfile. | 2013-03-28 | ||
| 63 | ERROR | Should iterate of ids and call Status.destroy for each. | |||
| 157 | 139 | SUGGEST | The code for the “download_tb” task in the torquebox init.pp includes a dependency on the torquebox user, but the first time you run it, on page 139, you haven’t added the task to create the Torquebox user yet. I suggest either completing the entire init.pp or removing this dependency from the code so that it will run in the order suggested in the book. (I tried to include the code here, but the form wouldn’t allow some part of it) | 2013-03-28 | |
| 129 | ERROR | Need to explicitly include the statuses.js | |||
| 171 | 153 | TYPO | The version of mod_cluster that you have the tar command for: Does not match the one you listed the download link for in the footnote: | 2013-03-28 | |
| 149 | ERROR | Stomp host needs to be reconfigured when using a cluster | |||
| 190 | 173 | SUGGEST | Modify Section 9.4 Creating a Management Bean to use torquebox instead of Trinidad. Half of the book is devoted to why TorqueBox is better and how to optimize it, so that should be the server you use to demonstrate MBeans. | ||
| 47 | ERROR | The code: sudo jgem install warbler-exec doesn’t work. The way jruby is installed means that it’s not on the PATH. In my local copy I’ve added some symlinks to the jruby puppet manifest: file { ‘/usr/local/bin/jruby’: file { ‘/usr/local/bin/jgem’: Which resolves the issue. | 2013-03-28 | ||
| 47 | ERROR | The code sudo jgem install warbler-exec doesn’t work. Changing it to sudo sh -c “PATH=$PATH:/opt/jruby/bin; jgem install warbler-exec” works, since it adds the jruby installation to the path in the sudo environment. | 2013-03-28 |