Revert #768

Author: cowtowncoder

pom.xml

@@ -46,9 +46,6 @@ com.fasterxml.jackson.core.*;version=${project.version}
     <packageVersion.dir>com/fasterxml/jackson/core/json</packageVersion.dir>
     <packageVersion.package>${project.groupId}.json</packageVersion.package>
 
-    <!-- Bnd annotations for generating extra OSGi metadata. -->
-    <version.bnd.annotation>6.3.1</version.bnd.annotation>
-
     <!-- for Reproducible Builds -->
     <project.build.outputTimestamp>2022-10-10T23:25:41Z</project.build.outputTimestamp>
   </properties>
@@ -147,16 +144,7 @@ com.fasterxml.jackson.core.*;version=${project.version}
   </build>
 
   <dependencies>
-    <!--  31-Jul-2022, tatu: [core#768] OSGi metadata -->
-    <dependency>
-      <groupId>biz.aQute.bnd</groupId>
-      <artifactId>biz.aQute.bnd.annotation</artifactId>
-      <version>${version.bnd.annotation}</version>
-      <optional>true</optional>
-      <scope>provided</scope>
-    </dependency>
-
-    <!--  plus test dependencies -->
+    <!-- Test dependencies -->
     <dependency>
       <groupId>org.junit.vintage</groupId>
       <artifactId>junit-vintage-engine</artifactId>

release-notes/VERSION-2.x

@@ -51,8 +51,6 @@ JSON library.
  (contributed by Evan G)
 #763: `JsonFactory.createParser()` with `File` may leak `InputStream`s
 #764: `JsonFactory.createGenerator()` with `File` may leak `OutputStream`s
-#768: Jackson bundles are missing OSGi's osgi.serviceloader metadata
- (requested by Chris R)
 #773: Add option to accept non-standard trailing decimal point
   (`JsonReadFeature.ALLOW_TRAILING_DECIMAL_POINT_FOR_NUMBERS`)
  (contributed by @pjfanning)
@@ -71,8 +69,6 @@ JSON library.
  (contributed by @pjfanning)
 #818: Calling `JsonPointer.compile(...)` on very deeply nested expression
   throws `StackOverflowErrror`
-#822: Declare osgi.serviceloader.registrar requirement as optional
- (contributed by Chris R)
 #828: Make `BigInteger` parsing lazy
  (contributed by @pjfanning)
 

src/main/java/com/fasterxml/jackson/core/JsonFactory.java

@@ -8,8 +8,6 @@
 import java.lang.ref.SoftReference;
 import java.net.URL;
 
-import aQute.bnd.annotation.spi.ServiceProvider;
-
 import com.fasterxml.jackson.core.format.InputAccessor;
 import com.fasterxml.jackson.core.format.MatchStrength;
 import com.fasterxml.jackson.core.io.*;
@@ -23,8 +21,6 @@
 import com.fasterxml.jackson.core.util.DefaultPrettyPrinter;
 import com.fasterxml.jackson.core.util.JacksonFeature;
 
-import static aQute.bnd.annotation.Resolution.OPTIONAL;
-
 /**
  * The main factory class of Jackson package, used to configure and
  * construct reader (aka parser, {@link JsonParser})
@@ -46,7 +42,6 @@
  * @author Tatu Saloranta
  */
 @SuppressWarnings("resource")
-@ServiceProvider(value = JsonFactory.class, resolution = OPTIONAL) // will be `TokenStreamFactory` in 3.x
 public class JsonFactory
     extends TokenStreamFactory
     implements Versioned,