Skip to content

Add compiled go2rtc to gitignore#1910

Merged
AlexxIT merged 1 commit intoAlexxIT:masterfrom
felipecrs:go2rtc-gitignore
Oct 14, 2025
Merged

Add compiled go2rtc to gitignore#1910
AlexxIT merged 1 commit intoAlexxIT:masterfrom
felipecrs:go2rtc-gitignore

Conversation

@felipecrs
Copy link
Contributor

@felipecrs felipecrs commented Oct 13, 2025

This is the standard output location when you compile go2rtc.

It should avoid issues like this.

@AlexxIT
Copy link
Owner

AlexxIT commented Oct 13, 2025

And it allows you to understand the programmer’s qualifications :)
An experienced programmer always looks at the files they commit, rather than blindly doing git add

@AlexxIT AlexxIT self-assigned this Oct 13, 2025
@felipecrs
Copy link
Contributor Author

felipecrs commented Oct 13, 2025

That is true. Lol

@felipecrs felipecrs changed the title Add go2rtc to gitignore Add compiled go2rtc to gitignore Oct 13, 2025
go2rtc_mac*
go2rtc_win*

/go2rtc
Copy link
Collaborator

Choose a reason for hiding this comment

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

/
?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

To only ignore go2rtc at the root of the repository.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because I don't want to ignore it elsewhere. Someone could create a sub-directory named go2rtc for whatever reason, and I don't want that to slip through.

@AlexxIT AlexxIT merged commit dae396a into AlexxIT:master Oct 14, 2025
@AlexxIT
Copy link
Owner

AlexxIT commented Oct 14, 2025

Thanks!

@AlexxIT AlexxIT added this to the v1.9.11 milestone Oct 21, 2025
@AlexxIT
Copy link
Owner

AlexxIT commented Oct 21, 2025

57Darling02 pushed a commit to 57Darling02/go2nvr that referenced this pull request Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants