Deprecate SingleStringAnnotation (#811)

It does not provide anything over NoTargetAnnotation. Its existence
suggests some significance so removing it for clarity.

2 files changed, 2 insertions, 1 deletions

diff --git a/src/main/scala/firrtl/ExecutionOptionsManager.scala b/src/main/scala/firrtl/ExecutionOptionsManager.scala
index c5f3506a..b359ba5e 100644
--- a/src/main/scala/firrtl/ExecutionOptionsManager.scala
+++ b/src/main/scala/firrtl/ExecutionOptionsManager.scala
@@ -69,7 +69,7 @@ case class CommonOptions(
 }
 
 /** Annotation that contains the [[CommonOptions]] target directory */
-case class TargetDirAnnotation(value: String) extends SingleStringAnnotation
+case class TargetDirAnnotation(value: String) extends NoTargetAnnotation
 
 trait HasCommonOptions {
   self: ExecutionOptionsManager =>
diff --git a/src/main/scala/firrtl/annotations/Annotation.scala b/src/main/scala/firrtl/annotations/Annotation.scala
index b16509a1..4b0591bf 100644
--- a/src/main/scala/firrtl/annotations/Annotation.scala
+++ b/src/main/scala/firrtl/annotations/Annotation.scala
@@ -51,6 +51,7 @@ trait SingleTargetAnnotation[T <: Named] extends Annotation {
     })).getOrElse(List(this))
 }
 
+@deprecated("Just extend NoTargetAnnotation", "1.1")
 trait SingleStringAnnotation extends NoTargetAnnotation {
   def value: String
 }