I'm new to this, but isn't the preprocessor folding patch retainable if we simply removed the pseudo:: namespace from the new function? The relevant header file was copied to support while clang-pseudo was removed.
Search Criteria
Package Details: clangd-opt-git 20.r12369.g0d38f64e7df9-1
Package Actions
Git Clone URL: | https://aur.archlinux.org/clangd-opt-git.git (read-only, click to copy) |
---|---|
Package Base: | clangd-opt-git |
Description: | Trunk version of standalone clangd binary, with custom patches (look AUR page or PKGBUILD comments) |
Upstream URL: | https://llvm.org/ |
Licenses: | Apache-2.0 WITH LLVM-exception |
Conflicts: | clangd-opt |
Provides: | clangd-opt |
Replaces: | clangd-opt |
Submitter: | sr.team |
Maintainer: | sr.team |
Last Packager: | sr.team |
Votes: | 2 |
Popularity: | 0.015243 |
First Submitted: | 2024-03-15 10:25 (UTC) |
Last Updated: | 2024-11-16 00:32 (UTC) |
Dependencies (18)
- gcc-libs (gcc-libs-gitAUR, gccrs-libs-gitAUR, gcc11-libsAUR, gcc-libs-snapshotAUR)
- glibc (glibc-gitAUR, glibc-linux4AUR, glibc-eacAUR, glibc-eac-binAUR, glibc-eac-rocoAUR)
- ncurses (ncurses-gitAUR)
- zlib (zlib-ng-compat-gitAUR, zlib-gitAUR, zlib-ng-compat)
- zstd (zstd-gitAUR, zstd-staticAUR)
- cmake (cmake-gitAUR) (make)
- git (git-gitAUR, git-glAUR) (make)
- libedit (make)
- libffi (libffi-gitAUR) (make)
- libxml2 (libxml2-gitAUR, libxml2-2.9AUR) (make)
- ncurses (ncurses-gitAUR) (make)
- ninja (ninja-kitwareAUR, ninja-memAUR, ninja-fuchsia-gitAUR, ninja-gitAUR, ninja-jobserverAUR) (make)
- patch (patch-gitAUR) (make)
- python-psutil (make)
- python-setuptools (make)
- python-sphinx (python-sphinx-gitAUR) (make)
- zlib (zlib-ng-compat-gitAUR, zlib-gitAUR, zlib-ng-compat) (make)
- zstd (zstd-gitAUR, zstd-staticAUR) (make)
Required by (0)
Sources (23)
- config-include-style.patch
- doxygen-extra-render-trunk.patch
- doxygen-more-fields.patch
- git+https://github.com/llvm/llvm-project.git#branch=main
- hover-bit-fields-mask.patch
- hover-doxygen-trunk.patch
- hover-field-idx.patch
- hover-hex-formats.patch
- hover-layout-everyhere.patch
- hover-no-defs.patch
- hover-record-paddings.patch
- hover-resolve-forward-params.patch
- hover-virt-offset.patch
- implement-abstract-class.patch
- inlay-hints-blockend-linelimit10.patch
- inlay-hints-hide-deduced-types-ignore-evident.patch
- inlay-hints-paddings.patch
- lsp-codelens.patch
- lsp-remove-files-from-cdb.patch
- postfix-completion-trunk.patch
- refactor-extract-function.patch
- resolve-depend-type.patch
- resolve-incomplete-header-includes.patch
aliu commented on 2024-10-05 03:07 (UTC)
sr.team commented on 2024-08-08 00:40 (UTC)
@aliu It appears that this issue occurred while resolving a merge conflict. Thank you for bringing it to my attention!
aliu commented on 2024-08-07 13:42 (UTC)
You're the (wonderful!) programmer here, but I do wonder about the point of the recent hotfix. Wouldn't redefining the variable ThisLocSkipped lead to a violation of the ODR (One-Definition Rule) when compiling?
aliu commented on 2024-04-20 14:59 (UTC)
Oops, well thanks for the links. I don't think we should have every package depend on glibc and gcc-libs though; the link you provided also says
In some cases this is not necessary and may or may not be listed, for example glibc cannot be uninstalled as every system needs some C library
I don't think requiring every package to add gcc-libs and glibc in depends is a good idea (and I think we all agree on not needing duplication between makedepends and depends). Anyways, I agree that zlib can be added back.
MarsSeed commented on 2024-04-20 14:53 (UTC)
@aliu, your recommendation is based on outdated guidelines.
Current guidelines say:
The depends array should list all direct first level dependencies even when some are already declared transitively.
Also, namcap, Arch's own package analysis tool complains if a package has ELF files that link to any other ELF file but don't declare those in the depends array - even if the libraries in question are gcc-libs or glibc.
Also as per above guidelines, zlib is needed in depends, even if zstd also links to it. namcap verifies this, as it also lists zlib as a needed direct dependency after its analysis.
@sr.team, please kindly add back gcc-libs, glibc and zlib to depends array, and bump pkgrel to 2 to ensure every user gets the updated PKGBUILD. Thank you in advance!
aliu commented on 2024-04-20 14:42 (UTC) (edited on 2024-04-20 14:43 (UTC) by aliu)
gcc-libs and glibc are in base and zlib is required by zstd, so the only dependencies this needs are ncurses and zstd. Also, dependencies in depends can be removed from makedepends.
sr.team commented on 2024-03-15 15:11 (UTC)
@aliu Thanks for help with package renaming
aliu commented on 2024-03-15 14:45 (UTC) (edited on 2024-03-15 14:46 (UTC) by aliu)
Replaces should be added to the new package, not the old one. The variable should also be outside functions.
aliu commented on 2024-03-15 12:36 (UTC)
During conflict, usually the older package gets uninstalled. Since the old package is nothing, replaced would handle this well.
sr.team commented on 2024-03-15 12:22 (UTC)
@aliu currently this package not marked as conflict, to allow update from older package. Old package installs nothing currently. I've seen this solution with migrating to vlc-git from vlc-wayland-git
Pinned Comments
sr.team commented on 2024-03-15 10:25 (UTC) (edited on 2024-09-29 14:12 (UTC) by sr.team)
This PKGBUILD build trunk version of clangd, and may apply some non-standard patches
Applying patches controls via enviroment variables (e.g.
CLANGD_DEFAULT_PATCH_STATE=y makepkg -si
):Toggle all below patches
CLANGD_DEFAULT_PATCH_STATE
:- 'n' - disable all patches
- 'y' - apply all patches
Apply user patches from
~/.config/clangd
. Patches must have extension*.patch
NOTE: This variable ignores the
CLANGD_DEFAULT_PATCH_STATE
variableCLANGD_USER_PATCHES
:- 'n' - do not use user patches
- 'y' - apply user patches (default)
Show Doxygen comments in hover (D134130)
CLANGD_DOXYGEN
:- 'n' - do not apply this patch
- 'y' - apply this patch
Resolve forwarded parameters in hover (D130265)
CLANGD_RESOLVEFWDPARAMS
:- 'n' - do not apply this patch
- 'y' - apply this patch
Implement textDocument/codeLens (D91930)
It's also required patched
vscode-clangd
extension - https://github.com/lightmelodies/vscode-clangd/commit/46266b4adb106b5080f5c771099c777806a16adcCLANGD_CODELENS
:- 'n' - do not apply this patch
- 'y' - apply this patch
Contextual postfix completions (D125224)
Demo: https://asciinema.org/a/rxyoicX44ya2jLDpdbNj3rVcC
CLANGD_POSTFIXCOMPLETION
:- 'n' - do not apply this patch
- 'y' - apply this patch
Extract Function: add hoisting support (D138499)
CLANGD_EXTRACTFUNC
:- 'n' - do not apply this patch
- 'y' - apply this patch
Add inlay hints for structure paddings
CLANGD_INLAYHINTSPADS
:- 'n' - do not apply this patch
- 'y' - apply this patch
Add hex formats in
size
andoffset
fields on hover, when values more than 10CLANGD_HOVERINHEX
:- 'n' - do not apply this patch
- 'y' - apply this patch
Show mask for bit fields on hover
Demo: Screen 1, Screen 2
CLANGD_HOVERBITFIELDSMASK
:- 'n' - do not apply this patch
- 'y' - apply this patch
Show offset of virtual methods
Limitations: supports only Itanium and MS abi
CLANGD_HOVERVIRTOFF
:- 'n' - do not apply this patch
- 'y' - apply this patch
Show layout hover info everyhere
CLANGD_HOVERLAYOUTEVERYHERE
:- 'n' - do not apply this patch
- 'y' - apply this patch
Hide definition from hover info for functions and variables
CLANGD_HOVERNODEFS
:- 'n' - do not apply this patch
- 'y' - apply this patch
Resolve the dependent type from its single instantiation (PR: 71279)
CLANGD_RESOLVEDEPTYPE
:- 'n' - do not apply this patch
- 'y' - apply this patch
Increase minimal limit for BlockEnd inlay-hints to 10 lines (default is 2)
This disabled show BlockEnd inlay-hints for small functions (less than 10 lines)
CLANGD_INLAYHINTSBLOCKEND
:- 'n' - do not apply this patch
- 'y' - apply this patch
Resolve includes in non-self-contained headers
Limitations: AST for source must be loaded before open incomplete header
CLANGD_RESOLVEINCHEADERS
:- 'n' - do not apply this patch
- 'y' - apply this patch
Add way to remove files from CDB via LSP
No need in many cases. Useful only in multi project workspaces with patched client
CLANGD_LSPREMOVEFROMCDB
:- 'n' - do not apply this patch
- 'y' - apply this patch
Show total record paddings in hover
CLANGD_HOVERRECORDPAD
:- 'n' - do not apply this patch
- 'y' - apply this patch
Allow specifying what headers are always included via "" or <> (PR: 67749)
CLANGD_CONFIG_INCLUDE_STYLE
:- 'n' - do not apply this patch
- 'y' - apply this patch
Add tweak for implementing abstract class (D94942)
CLANGD_IMPL_ABSTRACT_CLASS
:- 'n' - do not apply this patch
- 'y' - apply this patch
Hover information: Show fields indexes in layout:
Offset: 8 bytes (index 1)
CLANGD_HOVERFIELDIDX
:- 'n' - do not apply this patch
- 'y' - apply this patch
Inlay hints: Ignore evident types in deduced types
CLANGD_INLAYHINTSIGNOREEVIDENT
:- 'n' - do not apply this patch
- 'y' - apply this patch