Hi Arto, Bo,

Xiyue Deng <manp...@gmail.com> writes:

> Hi Bo,
>
> Bo YU <tsu.y...@gmail.com> writes:
>
>> Hi!
>>
>> On Wed, Oct 25, 2023 at 5:06 PM Arto Jantunen <vi...@debian.org> wrote:
>>>
>>> Xiyue Deng <manp...@gmail.com> writes:
>>>
>>> > Arto Jantunen <vi...@debian.org> writes:
>>> >
>>> >> Xiyue Deng <manp...@gmail.com> writes:
>>> >>
>>> >>> Hi Arto,
>>> >>>
>>> >>> Arto Jantunen <vi...@debian.org> writes:
>>> >>>
>>> >>>> Xiyue Deng <manp...@gmail.com> writes:
>>> >>>>> Package: sponsorship-requests
>>> >>>>> Severity: important
>>> >>>>> X-Debbugs-CC: debian-emac...@lists.debian.org
>>> >>>>>
>>> >>>>> Dear mentors,
>>> >>>>>
>>> >>>>> I am looking for a sponsor for my package "lsp-mode":
>>> >>>>>
>>> >>>>>  * Package name     : lsp-mode
>>> >>>>>    Version          : 8.0.0-6
>>> >>>>>    Upstream contact : Vibhav Pant <vibh...@gmail.com>
>>> >>>>>  * URL              : https://github.com/emacs-lsp/lsp-mode
>>> >>>>>  * License          : GPL-3+
>>> >>>>>  * Vcs              : https://salsa.debian.org/emacsen-team/lsp-mode
>>> >>>>>    Section          : lisp
>>> >>>>>
>>> >>>>> The source builds the following binary packages:
>>> >>>>>
>>> >>>>>   elpa-lsp-mode - Emacs client/library for the Language Server 
>>> >>>>> Protocol
>>> >>>>>
>>> >>>>> To access further information about this package, please visit the 
>>> >>>>> following URL:
>>> >>>>>
>>> >>>>>   https://mentors.debian.net/package/lsp-mode/
>>> >>>>>
>>> >>>>> Alternatively, you can download the package with 'dget' using this 
>>> >>>>> command:
>>> >>>>>
>>> >>>>>   dget -x 
>>> >>>>> https://mentors.debian.net/debian/pool/main/l/lsp-mode/lsp-mode_8.0.0-6.dsc
>>> >>>>>
>>> >>>>> Changes since the last upload:
>>> >>>>>
>>> >>>>>  lsp-mode (8.0.0-6) unstable; urgency=medium
>>> >>>>>  .
>>> >>>>>    * Add patch to fix test failures (Closes: #1052939).
>>> >>>>>    * Update Standards-Version to 4.6.2.  No change needed.
>>> >>>>>    * Add myself as uploader (Closes: #1042568).
>>> >>>>>    * Add signing key verification to d/watch.
>>> >>>>>    * Add d/upstream/metadata.
>>> >>>>>    * Add Upstream-Contact and update year in d/copyright.
>>> >>>>>    * Add patch to fix non-UTF-8 encoding.
>>> >>>>>    * Drop unused lintian overrides.
>>> >>>>
>>> >>>> Thanks for taking over this package.
>>> >>>>
>>> >>>> When I try to build this (under sbuild) I get the following build
>>> >>>> failure:
>>> >>>>
>>> >>>> Test ‘lsp-text-document-hover-request’ redefined
>>> >>>>
>>> >>>> Error: error ("Test ‘lsp-text-document-hover-request’ redefined")
>>> >>>>   mapbacktrace(#f(compiled-function (evald func args flags) #<bytecode 
>>> >>>> -0x187de6214517952>))
>>> >>>>   debug-early-backtrace()
>>> >>>>   debug-early(error (error "Test ‘lsp-text-document-hover-request’ 
>>> >>>> redefined"))
>>> >>>>   error("Test `%s' redefined" lsp-text-document-hover-request)
>>> >>>>   ert-set-test(lsp-text-document-hover-request #s(ert-test :name
>>> >>>> lsp-text-document-hover-request :documentation nil :body (closure (t) 
>>> >>>> nil
>>> >>>> (lsp-workspace-folders-add (f-join lsp-test-location "fixtures")) 
>>> >>>> (find-file
>>> >>>> (f-join lsp-test-location "fixtures/pyls/test.py")) (lsp) 
>>> >>>> (deferred:sync!
>>> >>>> (deferred:nextc (deferred:nextc (lsp-test--wait-for '(progn (eq 
>>> >>>> 'initialized
>>> >>>> (lsp--workspace-status (cl-first (lsp-workspaces)))))) #'(lambda (_) 
>>> >>>> (goto-char
>>> >>>> (point-min)) (search-forward "fn1") (lsp-def-request-async 
>>> >>>> "textDocument/hover"
>>> >>>> (lsp--text-document-position-params)))) #'(lambda (contents) (let* 
>>> >>>> ((fn-566
>>> >>>> #'lsp-hover?) (args-567 (condition-case err (let ((signal-hook-function
>>> >>>> #'ert--should-signal-hook)) (list contents)) (error (progn (setq fn-566
>>> >>>> #'signal) (list (car err) (cdr err))))))) (let ((value-568
>>> >>>> 'ert-form-evaluation-aborted-569)) (let (form-description-570) (if
>>> >>>> (unwind-protect (setq value-568 (apply fn-566 args-567)) (setq
>>> >>>> form-description-570 (nconc (list '(should (lsp-hover? contents))) 
>>> >>>> (list :form
>>> >>>> (cons fn-566 args-567)) (if (eql value-568 
>>> >>>> 'ert-form-evaluation-aborted-569) nil
>>> >>>> (list :value value-568)) (if (eql value-568 
>>> >>>> 'ert-form-evaluation-aborted-569)
>>> >>>> nil (let* ((-explainer- (and t (ert--get-explainer 'lsp-hover?)))) (if
>>> >>>> -explainer- (list :explanation (apply -explainer- args-567)) nil)))))
>>> >>>> (ert--signal-should-execution form-description-570)) nil (ert-fail
>>> >>>> form-description-570))) value-568))))) (kill-buffer)
>>> >>>> (lsp-workspace-folders-remove (f-join lsp-test-location "fixtures")))
>>> >>>> :most-recent-result nil :expected-result-type :passed :tags nil 
>>> >>>> :file-name
>>> >>>> "/<<PKGBUILDDIR>>/test/lsp-integration-test.el"))
>>> >>>>   
>>> >>>> load-with-code-conversion("/<<PKGBUILDDIR>>/test/lsp-integration-test.el"
>>> >>>>  "/<<PKGBUILDDIR>>/test/lsp-integration-test.el" nil t)
>>> >>>>   command-line-1(("-l" "package" "--eval" "(add-to-list 
>>> >>>> 'package-directory-list
>>> >>>> \"/usr/share/emacs/site-lisp/elpa\")" "--eval" "(add-to-list
>>> >>>> 'package-directory-list \"/usr/share/emacs/site-lisp/elpa-src\")" "-f"
>>> >>>> "package-initialize" "-L" "clients/" "-L" "." "-L" "test" "-l"
>>> >>>> "test/lsp-clangd-test.el" "-l" "test/lsp-completion-test.el" "-l"
>>> >>>> "test/lsp-file-watch-test.el" "-l" "test/lsp-integration-test.el" "-l"
>>> >>>> "test/lsp-io-test.el" "-l" "test/lsp-javascript-test.el" "-l"
>>> >>>> "test/lsp-methods-test.el" "-l" "test/lsp-mode-test.el" "-l"
>>> >>>> "test/lsp-protocol-test.el" "-l" "test/lsp-common-test.el" "-l"
>>> >>>> "debian/ert-helper.el"))
>>> >>>>   command-line()
>>> >>>>   normal-top-level()
>>> >>>> dh_elpa_test: error: emacs -batch -Q -l package --eval "(add-to-list
>>> >>>> 'package-directory-list \"/usr/share/emacs/site-lisp/elpa\")" --eval
>>> >>>> "(add-to-list 'package-directory-list 
>>> >>>> \"/usr/share/emacs/site-lisp/elpa-src\")"
>>> >>>> -f package-initialize -L clients/ -L . -L test -l 
>>> >>>> test/lsp-clangd-test.el -l
>>> >>>> test/lsp-completion-test.el -l test/lsp-file-watch-test.el -l
>>> >>>> test/lsp-integration-test.el -l test/lsp-io-test.el -l
>>> >>>> test/lsp-javascript-test.el -l test/lsp-methods-test.el -l 
>>> >>>> test/lsp-mode-test.el
>>> >>>> -l test/lsp-protocol-test.el -l test/lsp-common-test.el -l 
>>> >>>> debian/ert-helper.el
>>> >>>> returned exit code 255
>>> >>>> make: *** [debian/rules:4: binary] Error 25
>>> >>>> dpkg-buildpackage: error: debian/rules binary subprocess returned exit 
>>> >>>> status 2
>>> >>>>
>>> >>>> Is this something specific to my environment? I can't see two actual
>>> >>>> definitions of a test with that name...
>>> >>>
>>> >>> I tried downloading the dsc I uploaded to mentors and run sbuild on it
>>> >>> directly, but I cannot reproduce this issue after three tries.
>>> >>>
>>> >>> Can you also try to reproduce using the team repo?  It is just one
>>> >>> commit ("dch -r") away from the one on mentors.
>>> >>
>>> >> The team repo was the only thing I tried. I have never used mentors, and
>>> >> have no idea what to do with it.
>>> >
>>> > Ah I see.  As I haven been developing using the team repo git client I
>>> > can't reproduce there either.
>>> >
>>> > I presume your sbuild should be up-to-date.  Can you also try the dsc
>>> > from mentors?  The following should suffice.
>>> >
>>> > ,----
>>> > | $ dget -x 
>>> > https://mentors.debian.net/debian/pool/main/l/lsp-mode/lsp-mode_8.0.0-6.dsc
>>> > | $ sbuild lsp-mode_8.0.0-6.dsc
>>> > `----
>>>
>>> Fails in exactly the same way, in an up-to-date sid sbuild chroot.
>>
>> Same error here.
>
> Can you also send your build log for cross checking?  Also your
> ~/.sbuildrc if possible.  TIA.
>

For the unlikely but possible cause that tests with a long name is a
prefix of other tests that may trigger this issue, I have modified the
test name for testing purposes.  Can you help get the latest upload on
mentors and try again?  TIA.

,----
| $ dget -x 
https://mentors.debian.net/debian/pool/main/l/lsp-mode/lsp-mode_8.0.0-6.dsc
| $ sbuild lsp-mode_8.0.0-6.dsc
`----

P.S. If you can provide the failed build log and ~/.sbuildrc it may
still help to eliminate potential sbuild differences in our environment.

>>
>> BR,
>> Bo
>>>
>>> --
>>> Arto Jantunen
>>>

-- 
Xiyue Deng

Reply via email to