Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

doxygen: Add at v1.13.2 #578

Merged
merged 11 commits into from
Feb 3, 2025
Merged

doxygen: Add at v1.13.2 #578

merged 11 commits into from
Feb 3, 2025

Conversation

malfisya
Copy link
Contributor

@malfisya malfisya commented Jan 30, 2025 •
edited
Loading

Add doxygen at v1.13.2

@CLAassistant
Copy link

CLAassistant commented Jan 30, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@malfisya
Copy link
Contributor Author

Haven't rebuild it since created. Trying to rebuild it now.

@malfisya malfisya changed the title doxygen: Add at v1.12.0 doxygen: Add at v1.13.2 Jan 30, 2025
@malfisya
Copy link
Contributor Author

malfisya commented Jan 30, 2025 •
edited
Loading

The splitting part doesn't work

packages    :
    - "doxywizard":
          paths:
            - /usr/bin/doxywizard

Anything wrong with it?

EDIT: It works now, I did a typo before (path => paths)

@ReillyBrogan ReillyBrogan force-pushed the doxyviz2 branch 2 times, most recently from 49cdc0c to 0716eac Compare January 30, 2025 05:35
@ReillyBrogan
Copy link
Contributor

The splitting part doesn't work

packages    :
    - "doxywizard":
          paths:
            - /usr/bin/doxywizard

Anything wrong with it?

I dunno I built it as-is and it worked fine. I did adjust the whitespace on paths but it worked even before I did that.

Anyway, made a few tweaks like using binary() for flex/bison and moving the comment to the .patch file.

@malfisya malfisya marked this pull request as ready for review January 30, 2025 10:58
@malfisya malfisya requested a review from a team as a code owner January 30, 2025 10:58
malfisya and others added 11 commits January 30, 2025 16:51
@malfisya @ReillyBrogan
doxygen: Add at v1.13.2
3a8957a
@ReillyBrogan
cairomm: Build with doxygen
136e5b8 
Signed-off-by: Reilly Brogan <[email protected]>
@ReillyBrogan
glibmm: Build with doxygen
26d85aa 
Signed-off-by: Reilly Brogan <[email protected]>
@ReillyBrogan
gtkmm-4: Build with doxygen
b518217 
Signed-off-by: Reilly Brogan <[email protected]>
@ReillyBrogan
libmnl: Build with doxygen
3b4f10b 
Signed-off-by: Reilly Brogan <[email protected]>
@ReillyBrogan
libnetfilter_conntrack: Build with doxygen
d713eb6 
Signed-off-by: Reilly Brogan <[email protected]>
@ReillyBrogan
libnfnetlink: Build with doxygen
12b5d3f 
Signed-off-by: Reilly Brogan <[email protected]>
@ReillyBrogan
libnftnl: Build with doxygen
932138a 
Signed-off-by: Reilly Brogan <[email protected]>
@ReillyBrogan
libsigc++: Build with doxygen
cc86f73 
Signed-off-by: Reilly Brogan <[email protected]>
@ReillyBrogan
libxkbcommon: Remove docs note
0ea669b 
Signed-off-by: Reilly Brogan <[email protected]>
@ReillyBrogan
pangomm: Build with doxygen
0d09d49 
Signed-off-by: Reilly Brogan <[email protected]>
ikeycode
ikeycode approved these changes Feb 3, 2025
View reviewed changes
Copy link
Member

@ikeycode ikeycode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"damn son"

@ikeycode ikeycode merged commit 058a9c8 into serpent-os:main Feb 3, 2025
3 checks passed
@malfisya malfisya deleted the doxyviz2 branch February 3, 2025 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ikeycode ikeycode ikeycode approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@malfisya @CLAassistant @ReillyBrogan @ikeycode