Skip to content

Commit 0d4a6ae

Browse files
edsuedsu
committed
Use pyproject and remove setup.py
This commit includes changes to remove the use of setup.py and setuptools in favor of using the existing pyrproject.toml and hatch. A source layout was chosen, which required several changes such as: - pushing bagit.py into src/bagit/ - pushing locale into src/bagit/locale The gettext related code in setup.py was relocated to utils/locales.py It's possible the way that the gettext functionality is setup needs to adjustment, as I'm new to using it. Also we now rely on running tests with uv and pytest, which happens in the test github action. We could chose to update the PyPI publishing step to use uv build/publish too?
1 parent da04180 commit 0d4a6ae

File tree

13 files changed

+102
-103
lines changed

13 files changed

+102
-103
lines changed

‎.github/workflows/test.yml‎

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,17 +20,14 @@ jobs:
2020
strategy:
2121
fail-fast: false
2222
matrix:
23-
python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"]
23+
python-version: ["3.8", "3.9", "3.10", "3.11", "3.12", "3.13"]
2424
steps:
2525
- uses: actions/checkout@v4
2626
- name: Set up Python ${{ matrix.python-version }}
2727
uses: actions/setup-python@v5
2828
with:
2929
python-version: ${{ matrix.python-version }}
30-
- name: Install dependencies
31-
run: |
32-
python -m pip install --upgrade pip setuptools wheel
33-
pip install coverage
34-
pip install --editable .
35-
- name: Run test
36-
run: python -m unittest discover
30+
- name: Install uv
31+
uses: astral-sh/setup-uv@v5
32+
- name: Run tests
33+
run: uv run pytest

‎.gitignore‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,5 @@ dist
55
MANIFEST
66
bagit.egg-info
77
.idea
8+
uv.lock
9+
*.log

‎MANIFEST.in‎

Lines changed: 0 additions & 7 deletions
This file was deleted.

‎Makefile‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,13 @@ COMPILED_MESSAGES=$(patsubst %.po,%.mo, $(wildcard locale/*/LC_MESSAGES/bagit-py
33
all: messages compile
44

55
clean:
6-
rm -f locale/*/LC_MESSAGES/*.mo
6+
rm -f src/bagit/locale/*/LC_MESSAGES/*.mo
77

88
messages:
9-
xgettext --language=python -d bagit-python --no-location -o locale/bagit-python.pot bagit.py
9+
xgettext --language=python -d bagit-python --no-location -o src/bagit/locale/bagit-python.pot src/bagit/__init__.py
1010
# Until http://savannah.gnu.org/bugs/?20923 is fixed:
11-
sed -i '' -e 's/CHARSET/UTF-8/g' locale/bagit-python.pot
12-
msgmerge --no-fuzzy-matching --lang=en --output-file=locale/en/LC_MESSAGES/bagit-python.po locale/en/LC_MESSAGES/bagit-python.po locale/bagit-python.pot
11+
sed -i '' -e 's/CHARSET/UTF-8/g' src/bagit/locale/bagit-python.pot
12+
msgmerge --no-fuzzy-matching --lang=en --output-file=src/bagit/locale/en/LC_MESSAGES/bagit-python.po src/bagit/locale/en/LC_MESSAGES/bagit-python.po src/bagit/locale/bagit-python.pot
1313

1414
%.mo: %.po
1515
msgfmt -o $@ $<

‎pyproject.toml‎

Lines changed: 33 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,16 @@
11
[build-system]
2-
requires = ["setuptools>=64", "setuptools-scm>=8"]
3-
build-backend = "setuptools.build_meta"
2+
requires = ["hatchling", "hatch-vcs"]
3+
build-backend = "hatchling.build"
44

55
[project]
66
name = "bagit"
77
dynamic = ["version"]
88
description = "Create and validate BagIt packages"
99
readme = {file = "README.rst", content-type = "text/x-rst"}
1010
authors = [
11-
{ name = "Ed Summers", email = "[email protected]" },
11+
{ name = "Chris Adams", email = "[email protected]" },
12+
{ name = "Andy Boyko" },
13+
{ name = "Ed Summers", email = "[email protected]" }
1214
]
1315
classifiers = [
1416
"Intended Audience :: Developers",
@@ -18,6 +20,10 @@ classifiers = [
1820
"Topic :: Software Development :: Libraries :: Python Modules",
1921
"Topic :: System :: Filesystems",
2022
]
23+
requires-python = ">= 3.9"
24+
25+
[project.scripts]
26+
"bagit" = "bagit:main"
2127

2228
[project.urls]
2329
Homepage = "https://libraryofcongress.github.io/bagit-python/"
@@ -27,14 +33,34 @@ Homepage = "https://libraryofcongress.github.io/bagit-python/"
2733
[tool.ruff]
2834
target-version = "py38"
2935

30-
31-
[tool.setuptools_scm]
32-
3336
[tool.isort]
3437
line_length = 110
3538
default_section = "THIRDPARTY"
3639
known_first_party = "bagit"
3740

41+
[tool.pytest.ini_options]
42+
testpaths = ["test.py"]
43+
addopts = "-v"
44+
3845
[tool.coverage.run]
3946
branch = true
40-
include = "bagit.py"
47+
include = "src"
48+
49+
[tool.hatch.version]
50+
source = "vcs"
51+
52+
[tool.hatch.build.targets.sdist]
53+
packages = ["src/bagit"]
54+
include = [
55+
"src/bagit/*.py",
56+
"src/bagit/locale/*.po",
57+
"src/bagit/locale/*.mo",
58+
]
59+
60+
61+
[dependency-groups]
62+
dev = [
63+
"coverage>=7.8.2",
64+
"pytest>=8.4.0",
65+
"ruff>=0.11.12",
66+
]

‎setup.py‎

Lines changed: 0 additions & 61 deletions
This file was deleted.

bagit.py renamed to src/bagit/__init__.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -240,7 +240,7 @@ def make_bag(
240240
)
241241

242242
LOGGER.info(_("Creating bagit.txt"))
243-
txt = """BagIt-Version: 0.97\nTag-File-Character-Encoding: UTF-8\n"""
243+
txt = """BagIt-Version: 1.0\nTag-File-Character-Encoding: UTF-8\n"""
244244
with open_text_file("bagit.txt", "w") as bagit_file:
245245
bagit_file.write(txt)
246246

@@ -1481,10 +1481,7 @@ def _make_parser():
14811481

14821482
checksum_args = parser.add_argument_group(
14831483
_("Checksum Algorithms"),
1484-
_(
1485-
"Select the manifest algorithms to be used when creating bags"
1486-
" (default=%s)"
1487-
)
1484+
_("Select the manifest algorithms to be used when creating bags (default=%s)")
14881485
% ", ".join(DEFAULT_CHECKSUMS),
14891486
)
14901487

locale/bagit-python.pot renamed to src/bagit/locale/bagit-python.pot

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ msgid ""
88
msgstr ""
99
"Project-Id-Version: PACKAGE VERSION\n"
1010
"Report-Msgid-Bugs-To: \n"
11-
"POT-Creation-Date: 2018-06-26 10:28-0400\n"
11+
"POT-Creation-Date: 2025-06-05 12:14-0400\n"
1212
"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
1313
"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
1414
"Language-Team: LANGUAGE <[email protected]>\n"
@@ -226,20 +226,23 @@ msgid ""
226226
"%(found_byte_count)d bytes"
227227
msgstr ""
228228

229-
msgid "Bag validation failed"
229+
msgid "Bag is incomplete"
230230
msgstr ""
231231

232232
#, python-format
233233
msgid "Unable to calculate file hashes for %s"
234234
msgstr ""
235235

236+
msgid "Bag validation failed"
237+
msgstr ""
238+
236239
msgid "bagit.txt must not contain a byte-order mark"
237240
msgstr ""
238241

239242
#, python-format
240243
msgid ""
241-
"%(path)s %(algorithm)s validation failed: expected=\"%(expected)s\" found="
242-
"\"%(found)s\""
244+
"%(path)s %(algorithm)s validation failed: expected=\"%(expected)s\" "
245+
"found=\"%(found)s\""
243246
msgstr ""
244247

245248
#, python-format
@@ -345,16 +348,23 @@ msgstr ""
345348
msgid "bagit-python version %s"
346349
msgstr ""
347350

348-
msgid "The number of processes must be 0 or greater"
351+
msgid "The number of processes must be greater than 0"
349352
msgstr ""
350353

351354
msgid "--fast is only allowed as an option for --validate!"
352355
msgstr ""
353356

357+
msgid "--completeness-only is only allowed as an option for --validate!"
358+
msgstr ""
359+
354360
#, python-format
355361
msgid "%s valid according to Payload-Oxum"
356362
msgstr ""
357363

364+
#, python-format
365+
msgid "%s is complete and valid according to Payload-Oxum"
366+
msgstr ""
367+
358368
#, python-format
359369
msgid "%s is valid"
360370
msgstr ""

locale/bagit.pot renamed to src/bagit/locale/bagit.pot

File renamed without changes.

locale/en/LC_MESSAGES/bagit-python.po renamed to src/bagit/locale/en/LC_MESSAGES/bagit-python.po

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ msgid ""
77
msgstr ""
88
"Project-Id-Version: PACKAGE VERSION\n"
99
"Report-Msgid-Bugs-To: \n"
10-
"POT-Creation-Date: 2018-06-26 10:28-0400\n"
10+
"POT-Creation-Date: 2025-06-05 12:14-0400\n"
1111
"PO-Revision-Date: 2017-04-27 15:02-0400\n"
1212
"Last-Translator: Automatically generated\n"
1313
"Language-Team: none\n"
@@ -233,20 +233,23 @@ msgid ""
233233
"%(found_byte_count)d bytes"
234234
msgstr ""
235235

236-
msgid "Bag validation failed"
236+
msgid "Bag is incomplete"
237237
msgstr ""
238238

239239
#, python-format
240240
msgid "Unable to calculate file hashes for %s"
241241
msgstr "Unable to calculate file hashes for %s"
242242

243+
msgid "Bag validation failed"
244+
msgstr ""
245+
243246
msgid "bagit.txt must not contain a byte-order mark"
244247
msgstr ""
245248

246249
#, python-format
247250
msgid ""
248-
"%(path)s %(algorithm)s validation failed: expected=\"%(expected)s\" found="
249-
"\"%(found)s\""
251+
"%(path)s %(algorithm)s validation failed: expected=\"%(expected)s\" "
252+
"found=\"%(found)s\""
250253
msgstr ""
251254

252255
#, python-format
@@ -352,16 +355,23 @@ msgstr ""
352355
msgid "bagit-python version %s"
353356
msgstr ""
354357

355-
msgid "The number of processes must be 0 or greater"
358+
msgid "The number of processes must be greater than 0"
356359
msgstr ""
357360

358361
msgid "--fast is only allowed as an option for --validate!"
359362
msgstr ""
360363

364+
msgid "--completeness-only is only allowed as an option for --validate!"
365+
msgstr ""
366+
361367
#, python-format
362368
msgid "%s valid according to Payload-Oxum"
363369
msgstr "%s valid according to Payload-Oxum"
364370

371+
#, python-format
372+
msgid "%s is complete and valid according to Payload-Oxum"
373+
msgstr ""
374+
365375
#, python-format
366376
msgid "%s is valid"
367377
msgstr "%s is valid"

0 commit comments

Comments
 (0)