fix copypasta output issues in enable/disable actions

these were copied from "remount" without changing.
status thingy was completely useless or wrong

none of the subclasses return booleans here, most of the time
the last line is a debug log message, which might have a side
effect of returning true.  either the subclass does the thing
successfully or else it raises.
make umount/mount style remount possible

the subclasses have code that clearly wants to do this, but since
we never call remount_fs when the resource doesn't support remount
the code in remount_fs to handle when the resource doesn't support
remount is inaccessible code.
add back a bit of debugging

this is what the status lines were trying to do...
fix mount specs busted by refactoring
whitespace
spec file modernization
copyright update
update date in copyright
code review update

- add some comments on implementing subclass
- use accessors not instance variables
- poll instead of just sleeping
comment tweaks
comment tweak
moar comment tweaks
fix spec test

also drop the loop down to 0.1 sleep
don't infinite loop on umounting the filesystem
spec test the retry failure loop

- create an accessor for dependency injection
- make sure that it raises when it runs out of retries
add a note on mount provider cleanup
Merge pull request #1454 from opscode/lcg/mount_stuff

Fix some mount provider rage