[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: python-lsp-server new upstream version



On Fri, 2025-11-21 at 11:20 +0100, Jochen Sprickerhof wrote:
> 
> quoting form the wiki:
> 
> > Once you've built and uploaded your package,
> 
> Note the "uploaded" (to the Debian archive). So you should not tag.

I want to be extra careful to not disturb anyone, so I'd like to
clarify: is the upload to Debian archive done with 'git push origin : -
-tags'?

This is what the wiki says about the upload, which seems to be
equivalent:

"""

Uploading

Push your repo to salsa.debian.org: 

$ git push --set-upstream git@salsa.debian.org:python-
team/packages/<srcpkgname>.git : --tags
"""

And is the tagging that I should not do what the wiki says in the
tagging section?


$ gbp buildpackage --git-tag-only
$ git push --tags
> 
> Can you test if this is a test only dependency and can be annotated 
> with <!nocheck>?
> 

Good shout, thanks. websockets are in the [project.optional-
dependencies] in the pyproject.toml of the upstream, so I assume it
should be annotated with <!nocheck>.

> > 
> > +Build-Depends: black,
> 
> Dito here.

This one is in the project [dependencies] section in the
pyproject.toml. I'll leave as it is.

> 
> 
> 
> Please add the test in d/rules instead.
> 

The test has been disabled in the debian/rules.


Thank you for the comments Jochen, please let me know if spot anything
else.

Cheers,
Aeliton
From 86d447a3af5be27c58d3af9407a8c2c36d09c7c1 Mon Sep 17 00:00:00 2001
From: Aeliton Silva <aeliton@aeliton.com>
Date: Fri, 21 Nov 2025 05:15:39 -0300
Subject: [PATCH 7/7] Update changelog for 1.13.2-1 release

---
 debian/changelog | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 2c2b643..a4280cf 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,15 @@
+python-lsp-server (1.13.2-1) UNRELEASED; urgency=medium
+
+  * New upstream version 1.13.2
+  * Rediff patches
+  * Fix test to work with jedi < 0.19.2
+  * Add black as a build dependency
+  * Update to pyqt6
+  * Add python3-websockets as build dependency
+  * Skip faulty test
+
+ -- Aeliton Silva <aeliton@aeliton.com>  Fri, 21 Nov 2025 05:15:30 -0300
+
 python-lsp-server (1.12.0-3) unstable; urgency=medium
 
   * Increase test timeout
-- 
2.51.0

From ee3a2098a55ab94697a12ce2482d5ce6377af665 Mon Sep 17 00:00:00 2001
From: Aeliton Silva <aeliton@aeliton.com>
Date: Fri, 21 Nov 2025 17:53:23 -0300
Subject: [PATCH 6/7] Skip faulty test

---
 debian/rules | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/debian/rules b/debian/rules
index 69489e2..cc82f53 100755
--- a/debian/rules
+++ b/debian/rules
@@ -4,7 +4,7 @@
 undefine CI
 
 # test_per_file_caching and test_per_file_caching: fails during build
-export PYBUILD_TEST_ARGS = -k "not test_lint_free_pylint and not test_per_file_caching"
+export PYBUILD_TEST_ARGS = -k "not test_lint_free_pylint and not test_per_file_caching and not test_missing_message"
 
 %:
 	dh $@ --buildsystem=pybuild
-- 
2.51.0

From 38707ea46194042bce8d38b8c147dfd2d40b62a8 Mon Sep 17 00:00:00 2001
From: Aeliton Silva <aeliton@aeliton.com>
Date: Fri, 21 Nov 2025 05:14:56 -0300
Subject: [PATCH 5/7] Add python3-websockets as build dependency

---
 debian/control | 1 +
 1 file changed, 1 insertion(+)

diff --git a/debian/control b/debian/control
index 9d9d026..9e7019f 100644
--- a/debian/control
+++ b/debian/control
@@ -28,6 +28,7 @@ Build-Depends: black,
                python3-setuptools,
                python3-setuptools-scm,
                python3-ujson,
+               python3-websockets <!nocheck>,
                python3-whatthepatch (>= 1.0.2) <!nocheck>,
                python3-yapf <!nocheck>
 Standards-Version: 4.7.0
-- 
2.51.0

From dfb41f103e686e0e9fa9cdaf898c3e64557d3b05 Mon Sep 17 00:00:00 2001
From: Aeliton Silva <aeliton@aeliton.com>
Date: Fri, 21 Nov 2025 05:14:41 -0300
Subject: [PATCH 4/7] Update to pyqt6

---
 debian/control | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/debian/control b/debian/control
index e012904..9d9d026 100644
--- a/debian/control
+++ b/debian/control
@@ -22,7 +22,7 @@ Build-Depends: black,
                python3-pluggy (>= 1.0.0),
                python3-pydocstyle (>= 6.3.0) <!nocheck>,
                python3-pylsp-jsonrpc,
-               python3-pyqt5 <!nocheck>,
+               python3-pyqt6 <!nocheck>,
                python3-pytest <!nocheck>,
                python3-rope <!nocheck>,
                python3-setuptools,
-- 
2.51.0

From 839fef23622fc5dc4d163644176032ff4e8be2b2 Mon Sep 17 00:00:00 2001
From: Aeliton Silva <aeliton@aeliton.com>
Date: Fri, 21 Nov 2025 05:14:11 -0300
Subject: [PATCH 3/7] Add black as a build dependency

---
 debian/control | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/debian/control b/debian/control
index b052494..e012904 100644
--- a/debian/control
+++ b/debian/control
@@ -5,7 +5,8 @@ Maintainer: Debian Python Team <team+python@tracker.debian.org>
 Uploaders: Pablo Mestre Drake <pmdcuba@gmail.com>,
            Julian Gilbey <jdg@debian.org>,
            Jochen Sprickerhof <jspricke@debian.org>
-Build-Depends: debhelper-compat (= 13),
+Build-Depends: black,
+               debhelper-compat (= 13),
                dh-sequence-python3,
                flake8 <!nocheck>,
                pybuild-plugin-pyproject,
-- 
2.51.0

From d45cee16505ca14245fcfc56ee4b3a22d6ccfe37 Mon Sep 17 00:00:00 2001
From: Aeliton Silva <aeliton@aeliton.com>
Date: Fri, 21 Nov 2025 05:11:25 -0300
Subject: [PATCH 2/7] Fix test to work with jedi < 0.19.2

---
 ...03-Fix-test-to-work-with-jedi-0.19.1.patch | 26 +++++++++++++++++++
 debian/patches/series                         |  1 +
 2 files changed, 27 insertions(+)
 create mode 100644 debian/patches/0003-Fix-test-to-work-with-jedi-0.19.1.patch

diff --git a/debian/patches/0003-Fix-test-to-work-with-jedi-0.19.1.patch b/debian/patches/0003-Fix-test-to-work-with-jedi-0.19.1.patch
new file mode 100644
index 0000000..1545f87
--- /dev/null
+++ b/debian/patches/0003-Fix-test-to-work-with-jedi-0.19.1.patch
@@ -0,0 +1,26 @@
+From: Aeliton Silva <aeliton@aeliton.com>
+Date: Fri, 21 Nov 2025 01:34:55 -0300
+Subject: Fix test to work with jedi 0.19.1
+
+The change to jedi 0.19.2 sorts the completion list [1].
+
+The current jedi version in sid is 0.19.1.
+
+1. https://github.com/python-lsp/python-lsp-server/pull/609
+---
+ test/plugins/test_completion.py | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/test/plugins/test_completion.py b/test/plugins/test_completion.py
+index 015d0c4..ae5021f 100644
+--- a/test/plugins/test_completion.py
++++ b/test/plugins/test_completion.py
+@@ -181,7 +181,7 @@ def test_jedi_completion_with_fuzzy_enabled(config, workspace) -> None:
+ 
+     assert items
+ 
+-    expected = "isabs(s)"
++    expected = "commonprefix(m)" if JEDI_VERSION < "0.19.2" else "isabs(s)"
+     assert items[0]["label"] == expected
+ 
+     # Test we don't throw with big character
diff --git a/debian/patches/series b/debian/patches/series
index 583ee73..966f9c3 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,2 +1,3 @@
 0002-Disable-coverage-report.patch
 0002-Increase-test-timeout.patch
+0003-Fix-test-to-work-with-jedi-0.19.1.patch
-- 
2.51.0

From 026779c3fb436f4a1583c01feea48af479026620 Mon Sep 17 00:00:00 2001
From: Aeliton Silva <aeliton@aeliton.com>
Date: Fri, 21 Nov 2025 05:10:59 -0300
Subject: [PATCH 1/7] Rediff patches

---
 debian/patches/0002-Disable-coverage-report.patch | 4 ++--
 debian/patches/0002-Increase-test-timeout.patch   | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/debian/patches/0002-Disable-coverage-report.patch b/debian/patches/0002-Disable-coverage-report.patch
index 34dad47..f3e6030 100644
--- a/debian/patches/0002-Disable-coverage-report.patch
+++ b/debian/patches/0002-Disable-coverage-report.patch
@@ -7,10 +7,10 @@ Subject: Disable coverage report
  1 file changed, 1 deletion(-)
 
 diff --git a/pyproject.toml b/pyproject.toml
-index f9c6a52..7573d3e 100644
+index 0be4035..96ab95a 100644
 --- a/pyproject.toml
 +++ b/pyproject.toml
-@@ -180,7 +180,6 @@ write_to_template = "__version__ = \"{version}\"\n"  # VERSION_INFO is populated
+@@ -183,7 +183,6 @@ write_to_template = "__version__ = \"{version}\"\n"  # VERSION_INFO is populated
  
  [tool.pytest.ini_options]
  testpaths = ["test"]
diff --git a/debian/patches/0002-Increase-test-timeout.patch b/debian/patches/0002-Increase-test-timeout.patch
index bee8306..20626cf 100644
--- a/debian/patches/0002-Increase-test-timeout.patch
+++ b/debian/patches/0002-Increase-test-timeout.patch
@@ -8,7 +8,7 @@ Was failing on arm64 and i386
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/test/test_utils.py b/test/test_utils.py
-index 07d04e3..4983d29 100644
+index 7ed6214..1d8b25e 100644
 --- a/test/test_utils.py
 +++ b/test/test_utils.py
 @@ -16,7 +16,7 @@ from pylsp import _utils
@@ -19,4 +19,4 @@ index 07d04e3..4983d29 100644
 +CALL_TIMEOUT_IN_SECONDS = 60
  
  
- def send_notebook_did_open(client, cells: List[str]) -> None:
+ def send_notebook_did_open(client, cells: list[str]) -> None:
-- 
2.51.0


Reply to: