Page MenuHomeSoftware Heritage

Drop ctags indexer references and hidden api using it
ClosedPublic

Authored by ardumont on Aug 4 2022, 10:24 AM.

Details

Summary

The ctags indexer has been stopped a long time ago and recently got dropped from the
indexer base code. As the api exposing this in the webapp is hidden, we can safely
remove them.

This also fixes build [1].

Related to T4273

[1] https://jenkins.softwareheritage.org/view/swh%20master%20(draft)/job/DWAPPS/job/tests/3265/console

Diff Detail

Event Timeline

Build has FAILED

Patch application report for D8179 (id=29529)

Rebasing onto 6f223dcb66...

Current branch diff-target is up to date.
Changes applied before test
commit 89954bdce3389091d4bf4feeb188e241fa44fcd4
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Aug 4 10:22:56 2022 +0200

    tests/data: Drop ctags indexer reference
    
    They no longer exist.
    
    This fixes build [1]
    
    Related to T4273
    
    [1] https://jenkins.softwareheritage.org/view/swh%20master%20(draft)/job/DWAPPS/job/tests/3265/console

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1949/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1949/console

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 4 2022, 10:41 AM
Harbormaster failed remote builds in B30596: Diff 29529!

Fix furher ctags related code

ardumont retitled this revision from tests/data: Drop ctags indexer reference to tests/data: Drop ctags indexer references and hidden api using it.Aug 4 2022, 11:15 AM
ardumont edited the summary of this revision. (Show Details)
ardumont added a reviewer: anlambert.

Build has FAILED

Patch application report for D8179 (id=29532)

Rebasing onto 6f223dcb66...

Current branch diff-target is up to date.
Changes applied before test
commit 2490004a2cb032f229e75b6043c755e2a5edd3dc
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Aug 4 10:22:56 2022 +0200

    tests/data: Drop ctags indexer references and hidden api using it
    
    The ctags indexer has been stopped a long time ago and recently got dropped from the
    indexer base code. As the api exposing this in the webapp is hidden, we can safely
    remove them.
    
    This also fixes build [1].
    
    Related to T4273
    
    [1] https://jenkins.softwareheritage.org/view/swh%20master%20(draft)/job/DWAPPS/job/tests/3265/console

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1951/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1951/console

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 4 2022, 11:15 AM
Harbormaster failed remote builds in B30599: Diff 29532!

Build has FAILED

Patch application report for D8179 (id=29533)

Rebasing onto 6f223dcb66...

Current branch diff-target is up to date.
Changes applied before test
commit 578b3bd05098340fcf1e780bb41f5dbb5fa8c5a9
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Aug 4 10:22:56 2022 +0200

    tests/data: Drop ctags indexer references and hidden api using it
    
    The ctags indexer has been stopped a long time ago and recently got dropped from the
    indexer base code. As the api exposing this in the webapp is hidden, we can safely
    remove them.
    
    This also fixes build [1].
    
    Related to T4273
    
    [1] https://jenkins.softwareheritage.org/view/swh%20master%20(draft)/job/DWAPPS/job/tests/3265/console

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1952/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1952/console

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 4 2022, 11:34 AM
Harbormaster failed remote builds in B30600: Diff 29533!
ardumont retitled this revision from tests/data: Drop ctags indexer references and hidden api using it to Drop ctags indexer references and hidden api using it.Aug 4 2022, 11:48 AM

Build is green

Patch application report for D8179 (id=29534)

Rebasing onto 6f223dcb66...

Current branch diff-target is up to date.
Changes applied before test
commit 3e5d552a328353fafa505904c664486c4ad2b05c
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Aug 4 10:22:56 2022 +0200

    Drop ctags indexer references and hidden api using it
    
    The ctags indexer has been stopped a long time ago and recently got dropped from the
    indexer base code. As the api exposing this in the webapp is hidden, we can safely
    remove them.
    
    This also fixes build [1].
    
    Related to T4273
    
    [1] https://jenkins.softwareheritage.org/view/swh%20master%20(draft)/job/DWAPPS/job/tests/3265/console

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1953/ for more details.

This revision is now accepted and ready to land.Aug 4 2022, 12:31 PM