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

refact: waydroid setup #915

Merged
merged 1 commit into from
Nov 11, 2024
Merged

refact: waydroid setup #915

merged 1 commit into from
Nov 11, 2024

Conversation

jeevithakannan2
Copy link
Contributor

@jeevithakannan2 jeevithakannan2 commented Nov 5, 2024 •
edited
Loading

Type of Change

  • Bug fix

Description

  • Move the headers installation outside of dkms if block so that headers can be installed even if dkms is installed.

Testing

  • Tested with no issues.

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • My changes generate no errors/warnings/merge conflicts.

@adamperkowski adamperkowski changed the title fix: waydroid setup headers not installing refact: waydroid setup Nov 5, 2024
@adamperkowski adamperkowski added bug Something isn't working refactor labels Nov 5, 2024
@ChrisTitusTech ChrisTitusTech merged commit c4ad5a9 into ChrisTitusTech:main Nov 11, 2024
4 checks passed
jeevithakannan2 added a commit to jeevithakannan2/linutil that referenced this pull request Nov 11, 2024
@jeevithakannan2
fix: headers not installing (ChrisTitusTech#915)
ac4cc2f
@jeevithakannan2 jeevithakannan2 deleted the waydroid branch November 17, 2024 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adamperkowski adamperkowski adamperkowski approved these changes

@ChrisTitusTech ChrisTitusTech ChrisTitusTech approved these changes

Assignees
No one assigned
Labels
bug Something isn't working refactor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jeevithakannan2 @ChrisTitusTech @adamperkowski