Skip to content

Multiple fixes #741

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
Merged
lzwind merged 3 commits into from
Oct 11, 2025
Merged

Multiple fixes #741

lzwind merged 3 commits into from
Oct 11, 2025

Conversation

@hillwoodroc
Copy link
Contributor

@hillwoodroc hillwoodroc commented Oct 10, 2025

  • fix: update function signature for GCC 15 compatibility-
  • Fix:(libcam_encoder) Add fallback definitions for FF_PROFILE_* macros
  • feat: Support ffmepg 7

@deepin-ci-robot
Copy link

deepin-ci-robot commented Oct 10, 2025

Hi @hillwoodroc. Thanks for your PR.

I'm waiting for a linuxdeepin member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

lzwind
lzwind reviewed Oct 11, 2025
View reviewed changes
3rdparty/libcam/libcam_encoder/encoder.c.orig Outdated
/*******************************************************************************#
# guvcview http://guvcview.sourceforge.net #
# #
# Paulo Assis <pj.assis@gmail.com> #
Copy link
Contributor

@lzwind lzwind Oct 11, 2025

Choose a reason for hiding this comment

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

This file should be a temporary file and it is recommended to delete it

Copy link
Contributor Author

@hillwoodroc hillwoodroc Oct 11, 2025

Choose a reason for hiding this comment

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

OK, have removed it.

@hillwoodroc
feat: Support ffmepg 7
7f61a39 
Log: Fix build on ffmepg 7
@hillwoodroc
fix:(libcam_encoder) Add fallback definitions for FF_PROFILE_* macros
4a8563f 
Some older FFmpeg or libav versions do not define FF_PROFILE_* macros.
Add conditional definitions to ensure compatibility with AV_PROFILE_*
constants, fixing build errors on systems with outdated headers.

Log:
@hillwoodroc
fix: update function signature for GCC 15 compatibility
561ce66 
The function 'v4l2core_check_device_list_events' signature was updated
to accept 'v4l2_dev_t *' parameter instead of having no parameters.

Log: This change addresses a breaking compatibility issue introduced in GCC 15,
which enforces stricter type checking on function declarations and definitions.

Previously, GCC allowed mismatched function declarations and definitions
when parameter types differ but ultimately point to the same struct type,
such as between 'struct _v4l2_dev_t *' and 'v4l2_dev_t *'.

GCC 15's change (see GCC 15 release notes) now treats such mismatches as errors,
requiring declarations and definitions to exactly match in parameter types.

This patch aligns the declaration with the definition to fix the
'conflicting types' compilation error when building with GCC 15
@deepin-ci-robot
Copy link

deepin-ci-robot commented Oct 11, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: hillwoodroc, lzwind

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@lzwind lzwind merged commit c8753e4 into linuxdeepin:release/eagle Oct 11, 2025
5 of 6 checks passed
add-uos pushed a commit to add-uos/deepin-screen-recorder that referenced this pull request Dec 17, 2025
@hillwoodroc @add-uos
Multiple fixes (linuxdeepin#741)
06553d9 
* feat: Support ffmepg 7

Log: Fix build on ffmepg 7

* fix:(libcam_encoder) Add fallback definitions for FF_PROFILE_* macros

Some older FFmpeg or libav versions do not define FF_PROFILE_* macros.
Add conditional definitions to ensure compatibility with AV_PROFILE_*
constants, fixing build errors on systems with outdated headers.

Log:

* fix: update function signature for GCC 15 compatibility

The function 'v4l2core_check_device_list_events' signature was updated
to accept 'v4l2_dev_t *' parameter instead of having no parameters.

Log: This change addresses a breaking compatibility issue introduced in GCC 15,
which enforces stricter type checking on function declarations and definitions.

Previously, GCC allowed mismatched function declarations and definitions
when parameter types differ but ultimately point to the same struct type,
such as between 'struct _v4l2_dev_t *' and 'v4l2_dev_t *'.

GCC 15's change (see GCC 15 release notes) now treats such mismatches as errors,
requiring declarations and definitions to exactly match in parameter types.

This patch aligns the declaration with the definition to fix the
'conflicting types' compilation error when building with GCC 15
add-uos pushed a commit to add-uos/deepin-screen-recorder that referenced this pull request Dec 17, 2025
@hillwoodroc @add-uos
Multiple fixes (linuxdeepin#741)
2897b56 
* feat: Support ffmepg 7

Log: Fix build on ffmepg 7

* fix:(libcam_encoder) Add fallback definitions for FF_PROFILE_* macros

Some older FFmpeg or libav versions do not define FF_PROFILE_* macros.
Add conditional definitions to ensure compatibility with AV_PROFILE_*
constants, fixing build errors on systems with outdated headers.

Log:

* fix: update function signature for GCC 15 compatibility

The function 'v4l2core_check_device_list_events' signature was updated
to accept 'v4l2_dev_t *' parameter instead of having no parameters.

Log: This change addresses a breaking compatibility issue introduced in GCC 15,
which enforces stricter type checking on function declarations and definitions.

Previously, GCC allowed mismatched function declarations and definitions
when parameter types differ but ultimately point to the same struct type,
such as between 'struct _v4l2_dev_t *' and 'v4l2_dev_t *'.

GCC 15's change (see GCC 15 release notes) now treats such mismatches as errors,
requiring declarations and definitions to exactly match in parameter types.

This patch aligns the declaration with the definition to fix the
'conflicting types' compilation error when building with GCC 15
add-uos pushed a commit to add-uos/deepin-screen-recorder that referenced this pull request Dec 17, 2025
@hillwoodroc @add-uos
fix: Multple fixes (linuxdeepin#741)
e80b253 
* feat: Support ffmepg 7

Log: Fix build on ffmepg 7

* fix:(libcam_encoder) Add fallback definitions for FF_PROFILE_* macros

Some older FFmpeg or libav versions do not define FF_PROFILE_* macros.
Add conditional definitions to ensure compatibility with AV_PROFILE_*
constants, fixing build errors on systems with outdated headers.

Log:

* fix: update function signature for GCC 15 compatibility

The function 'v4l2core_check_device_list_events' signature was updated
to accept 'v4l2_dev_t *' parameter instead of having no parameters.

Log: This change addresses a breaking compatibility issue introduced in GCC 15,
which enforces stricter type checking on function declarations and definitions.

Previously, GCC allowed mismatched function declarations and definitions
when parameter types differ but ultimately point to the same struct type,
such as between 'struct _v4l2_dev_t *' and 'v4l2_dev_t *'.

GCC 15's change (see GCC 15 release notes) now treats such mismatches as errors,
requiring declarations and definitions to exactly match in parameter types.

This patch aligns the declaration with the definition to fix the
'conflicting types' compilation error when building with GCC 15
deepin-bot bot pushed a commit that referenced this pull request Dec 17, 2025
@hillwoodroc @deepin-bot
fix: Multple fixes (#741)
2293663 
* feat: Support ffmepg 7

Log: Fix build on ffmepg 7

* fix:(libcam_encoder) Add fallback definitions for FF_PROFILE_* macros

Some older FFmpeg or libav versions do not define FF_PROFILE_* macros.
Add conditional definitions to ensure compatibility with AV_PROFILE_*
constants, fixing build errors on systems with outdated headers.

Log:

* fix: update function signature for GCC 15 compatibility

The function 'v4l2core_check_device_list_events' signature was updated
to accept 'v4l2_dev_t *' parameter instead of having no parameters.

Log: This change addresses a breaking compatibility issue introduced in GCC 15,
which enforces stricter type checking on function declarations and definitions.

Previously, GCC allowed mismatched function declarations and definitions
when parameter types differ but ultimately point to the same struct type,
such as between 'struct _v4l2_dev_t *' and 'v4l2_dev_t *'.

GCC 15's change (see GCC 15 release notes) now treats such mismatches as errors,
requiring declarations and definitions to exactly match in parameter types.

This patch aligns the declaration with the definition to fix the
'conflicting types' compilation error when building with GCC 15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lzwind lzwind lzwind approved these changes

Assignees

No one assigned

Labels

needs-ok-to-test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hillwoodroc @deepin-ci-robot @lzwind