Kemia kemia → 0.9 backlog

Comments and changes to this ticket

• 

  paul (at wingu) August 26th, 2010 @ 03:46 PM

  imported from github issues:
  To reproduce on my Mac running Snow Leopard:
  

  Richard-Wests-MacBook-Pro:kemia-testing rwest$ git clone http://github.com/rwest/kemia.git
  Cloning into kemia...
  remote: Counting objects: 3908, done.
  remote: Compressing objects: 100% (1154/1154), done.
  remote: Total 3908 (delta 2624), reused 3908 (delta 2624)
  Receiving objects: 100% (3908/3908), 31.89 MiB | 46 KiB/s, done.
  Resolving deltas: 100% (2624/2624), done.
  Richard-Wests-MacBook-Pro:kemia-testing rwest$ cd kemia/
  Richard-Wests-MacBook-Pro:kemia rwest$ open index.html 
  

  Then in Safari Develop > Show Error Console which reports:
  

  deps.js Failed to load resource: The requested URL was not found on this server.
  images/button-bg.png Failed to load resource: The requested URL was not found on this server.
  

  I notice that the html just has these scripts in the head:
  

  <script src="build/kemia-whitespace-only.js"></script>
  <script src="kemia/io/json_test_rxn.js"></script>
  <script src="kemia/io/io_test_molsample2.js"></script>
  

  but if I inspect the DOM of the page there's an additional:
  

  <script type="text/javascript" src="deps.js"></script>
  

  immediately after the line:
  

  <script src="build/kemia-whitespace-only.js"></script>
  

  Presumably it's coming from this line of kemia-whitespace-only.js:
  

  if(!goog.global.CLOSURE_NO_DEPS)goog.writeScriptTag_(goog.basePath+"deps.js")}
  

  hence my comment about goog.basePath="" above.

• 

  paul (at wingu) August 26th, 2010 @ 03:46 PM

  Thanks for the detailed explanation. I am able to reproduce in Safari and see the error message. You can also see the error when you load http://kemia.github.com/ It shows up in Google Chrome and Firefox too, but it is not so obvious as it is in Safari. This should not have any negative impact on running Kemia, other than the console error message.

  I think the cause is that our build script is trying to include deps.js when it should not. I will take a look at it.

  Thanks!

• 

  paul (at wingu) September 9th, 2010 @ 05:06 PM

  • Assigned user set to “paul (at wingu)”
  • Milestone order changed from “4” to “0”
• 

  paul (at wingu) September 17th, 2010 @ 12:49 PM

  • Milestone changed from “0.1.0 Clean up user interaction and rendering bugs” to “0.9 backlog”
  • Milestone order changed from “2” to “0”
• 

  paul (at wingu) October 6th, 2010 @ 01:21 PM

  • State changed from “new” to “resolved”

  fixed on reaction editor demo page