Best to move the default service_name from above down into a new else block here. It'll work fine as it is, since we just override that value, but it's a little confusing when we set the attribute more than once in a resource.

```
when "smartos"
  service_name "tomcat"
  supports :restart => true, :reload => false, :status => true
else
  service_name "tomcat#{node["tomcat"]["base_version"]}"
end
```