Permalink
Browse files

Please verify that this is the correct change with a test. I'm skepti…

…cal.

Revert "Fix symlink for last_assets"

This reverts commit abde30b.
  • Loading branch information...
Wilson Bilkovich
Wilson Bilkovich committed Jan 17, 2012
1 parent de7c16d commit fc64b50212b2e3efab652933f15e1e030aab5a88
Showing with 1 addition and 1 deletion.
  1. +1 −1 lib/engineyard-serverside/rails_asset_support.rb
@@ -84,7 +84,7 @@ def keep_existing_assets
else
mkdir -p #{current} #{last_asset_path};
fi;
-ln -nfs #{last_asset_path} #{c.release_path}/public
+ln -nfs #{current} #{last_asset_path} #{c.release_path}/public
COMMAND
end

3 comments on commit fc64b50

Contributor

martinemde replied Jan 17, 2012

Hmm, I suppose it should have a test. What does 3 argument ln -s do anyway?

Contributor

wilson replied Jan 18, 2012

As with the usual CMD SRC [...] DST pattern (e.g. rsync, cp, scp, etc) it links all the leading arguments into the final one. In this case, it is making symlinks called 'assets' and 'last_assets' in the 'public' dir. I suspect if we have a problem it is a mismatch between -serverside and what we did in nginx.conf to add support for this, rather than with this command.

Contributor

martinemde replied Jan 18, 2012

Ok, my mistake for letting this one go by. Let's dig around with this further and I'll reopen the issue that triggered it. #25 I believe.

Please sign in to comment.