"Look but don't touch" or "Look but don't profit from it" -license. Is there anything wrong with those? Being able to see the source-code is a great feature.

"Look but don't touch" or "Look but don't profit from it" -license. Is there anything wrong with those? Being able to see the source-code is a great feature. Being able to modify it even if you then can not sell it for profit is also great.