question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Remove top-level sstruct and xmlWriter?

See original GitHub issue

Now that extra_path=FontTools is gone from our setup.py (see https://github.com/fonttools/fonttools/pull/695), I would like to use this opportunity to also drop support for the stand-alone sstruct and xmlWriter modules that we still install to site-packages alongside the main fontTools package.

These were at some point moved by Behdad to their current location, inside fontTools.misc package, but they were subsequently restored to their original position (as aliases) after some users started complaining about sudden ImportErrors.

I think we can now safely drop those, as compatibility with the “sourceforge” fonttools is no longer so important.

Anybody who still relies on importing from those instead of the fontTools.misc.* counterparts, could keep using version 3.1.2. But future versions would no longer have them.

WDYT?

/cc @justvanrossum @LettError @typemytype @typesupply @benkiel

Issue Analytics

  • State:closed
  • Created 7 years ago
  • Comments:12 (6 by maintainers)

github_iconTop GitHub Comments

1reaction
anthrotypecommented, Oct 8, 2016

Well, my point of view is that ideally fontTools as a library should be self-contained in its own package. There might be historical reasons why sstruct and xmlWriter were put outside of the core fontTools package. But now I think we could all agree on moving those inside it for good. It’s not such a big deal to change one line in the code, no?

0reactions
anthrotypecommented, Dec 4, 2016

I’m going to proceed removing the top-level sstruct and xmlWriter aliases in the 3.3.0 release in the next few days, if nobody has objections.

Read more comments on GitHub >

github_iconTop Results From Across the Web

How to replace or remove the spl character in xml
Any help ? This is the Answer to help me to get rid of the spl char XmlWriter xmlWrite; XmlWriterSettings settings = new...
Read more >
matlab-xunit/xml_write.m at main - GitHub
tree, then saves it to XML file 'filename' using Matlab's xmlwrite. % function. ... RootName String with XML tag name used for root...
Read more >
Remove elements, attributes, and nodes from an XML tree
Learn how to remove elements, attributes, and other types of nodes from an XML tree.
Read more >
A Thorough Guide to Bond for C# - Microsoft Open Source
Changing of inheritance hierarchy (add/remove/substituting base struct); Changing between required and optional directly; Changing the default value of a field ...
Read more >
Struct LocalDate | Noda Time
LocalDate is an immutable struct representing a date within the calendar, ... See the top-level type documentation for more information about comparisons.
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found