Merge pull request #1007 from googlefonts/font-lib-copy

Support writing lib to .designspace and .fontra, make copyFont() copy lib
googlefonts · Dec 2, 2023 · 4525fe1 · 4525fe1
2 parents 5570e7c + aeff093
commit 4525fe1
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/src/fontra/backends/copy.py b/src/fontra/backends/copy.py
@@ -14,6 +14,7 @@ async def copyFont(
     sourceBackend, destBackend, *, glyphNames=None, numTasks=1, progressInterval=0
 ):
     await destBackend.putGlobalAxes(await sourceBackend.getGlobalAxes())
+    await destBackend.putFontLib(await sourceBackend.getFontLib())
     glyphMap = await sourceBackend.getGlyphMap()
     glyphNamesInFont = set(glyphMap)
     glyphNamesToCopy = sorted(

diff --git a/src/fontra/backends/designspace.py b/src/fontra/backends/designspace.py
@@ -561,7 +561,11 @@ async def getUnitsPerEm(self):
         return self.defaultFontInfo.unitsPerEm
 
     async def getFontLib(self):
-        return self.dsDoc.lib
+        return deepcopy(self.dsDoc.lib)
+
+    async def putFontLib(self, lib):
+        self.dsDoc.lib = deepcopy(lib)
+        self.dsDoc.write(self.dsDoc.path)
 
     async def watchExternalChanges(self):
         ufoPaths = sorted(set(self.ufoLayers.iterAttrs("path")))

diff --git a/src/fontra/backends/fontra.py b/src/fontra/backends/fontra.py
@@ -96,7 +96,11 @@ async def putGlobalAxes(self, axes):
         self._scheduler.schedule(self._writeFontData)
 
     async def getFontLib(self):
-        return {}
+        return deepcopy(self.fontData.lib)
+
+    async def putFontLib(self, lib):
+        self.fontData.lib = deepcopy(lib)
+        self._scheduler.schedule(self._writeFontData)
 
     def _readGlyphInfo(self):
         with open(self.glyphInfoPath, "r", encoding="utf-8") as file: