fixes pkgcheck output "UnderscoreInUseFlag" #359

Merged
SlavekB merged 1 commits from issue/UnderscoreInUseFlag into master 3 months ago
G-Src commented 3 months ago
Collaborator

pkgcheck scan:

trinity-apps/k3b
  UnderscoreInUseFlag: USE flag 'ffmpeg_all_codecs' uses reserved underscore character
pkgcheck scan: ``` trinity-apps/k3b UnderscoreInUseFlag: USE flag 'ffmpeg_all_codecs' uses reserved underscore character ```
G-Src added 1 commit 3 months ago
1f750bb6b2
fixes pkgcheck output
G-Src added the PR/rfc label 3 months ago
SlavekB reviewed 3 months ago
SlavekB left a comment
Owner

That seems good to me. Only there is a question – see below.
Please, can you make rebase to the current head so that your GPG signatures can be preserved while merging?

That seems good to me. Only there is a question – see below. Please, can you make rebase to the current head so that your GPG signatures can be preserved while merging?
# Copyright 1999-2020 Gentoo Authors
# Copyright 2020 The Trinity Desktop Project
# Copyright 1999-2023 Gentoo Authors
# Copyright 2020-2023 The Trinity Desktop Project
Owner

Is it time to move it to 2024?

Is it time to move it to 2024?
G-Src commented 3 months ago
Poster
Collaborator

yeah sure

yeah sure
G-Src commented 3 months ago
Poster
Collaborator

I can't 'git push', there seems to be something wrong with my network.. please wait for a few minutes. I'm really sorry.

I can't 'git push', there seems to be something wrong with my network.. please wait for a few minutes. I'm really sorry.
G-Src marked this conversation as resolved
Collaborator

I can't 'git push', there seems to be something wrong with my network.. please wait for a few minutes. I'm really sorry.

-f --force

> I can't 'git push', there seems to be something wrong with my network.. please wait for a few minutes. I'm really sorry. `-f` `--force`
G-Src commented 3 months ago
Poster
Collaborator

I can't 'git push', there seems to be something wrong with my network.. please wait for a few minutes. I'm really sorry.

-f --force

yes i know there's '-f' flag.

> > I can't 'git push', there seems to be something wrong with my network.. please wait for a few minutes. I'm really sorry. > > `-f` `--force` > yes i know there's '-f' flag.
G-Src force-pushed issue/UnderscoreInUseFlag from 1f750bb6b2 to 850b6fdb2e 3 months ago
G-Src force-pushed issue/UnderscoreInUseFlag from 850b6fdb2e to eb8794e579 3 months ago
SlavekB approved these changes 3 months ago
SlavekB left a comment
Owner

All looks good.

All looks good.
SlavekB merged commit eb8794e579 into master 3 months ago
SlavekB deleted branch issue/UnderscoreInUseFlag 3 months ago
Collaborator

If you're interested, here's the link.
We haven't used this yet, but anything can happen.

If you're interested, here's the [link](https://mail.trinitydesktop.org/mailman3/postorius/lists/team-gentoo.trinitydesktop.org/). We haven't used this yet, but anything can happen.
SlavekB added this to the R14.1.2 release milestone 3 months ago
G-Src commented 3 months ago
Poster
Collaborator

If you're interested, here's the link.
We haven't used this yet, but anything can happen.

I'm not really familiar with mailing lists so I'll just stick with Gitea for a while.

> If you're interested, here's the [link](https://mail.trinitydesktop.org/mailman3/postorius/lists/team-gentoo.trinitydesktop.org/). > We haven't used this yet, but anything can happen. I'm not really familiar with mailing lists so I'll just stick with Gitea for a while.
G-Src requested review from SlavekB 3 months ago
G-Src commented 3 months ago
Poster
Collaborator

oops sorry i pressed the wrong button

oops sorry i pressed the wrong button
SlavekB approved these changes 3 months ago
SlavekB left a comment
Owner

All still looks good 😉

All still looks good 😉
Collaborator

One more thing... it looks like somewhere between squashes you mangled copyright notices a bit: AFAIK you shouldn't remove the starting year...

One more thing... it looks like somewhere between squashes you mangled copyright notices a bit: AFAIK you shouldn't remove the starting year...
G-Src commented 3 months ago
Poster
Collaborator

One more thing... it looks like somewhere between squashes you mangled copyright notices a bit: AFAIK you shouldn't remove the starting year...

https://gitweb.gentoo.org/repo/gentoo.git/tree/header.txt

> One more thing... it looks like somewhere between squashes you mangled copyright notices a bit: AFAIK you shouldn't remove the starting year... https://gitweb.gentoo.org/repo/gentoo.git/tree/header.txt
G-Src commented 3 months ago
Poster
Collaborator

header.txt doesn't have the beginning year so i removed the beginning year too. should i keep it?

header.txt doesn't have the beginning year so i removed the beginning year too. should i keep it?
Collaborator

header.txt doesn't have the beginning year so i removed the beginning year too. should i keep it?

The header.txt file has nothing to do with it at all.
These are the years when this package began to be distributed, the top date is when in Gentoo, and the bottom date is when in TDE. You should know that this ebuild is based on earlier Gentoo ebuilds. These dates should be corrected and the start year should be present. I am attaching an old ebuild from Gentoo.
I wouldn’t touch this package at all, since although it’s going to be put together, it’s not working. When trying to read the disk information it just freezes.
You should keep an eye on the dates, as they are added for a reason.

> header.txt doesn't have the beginning year so i removed the beginning year too. should i keep it? The header.txt file has nothing to do with it at all. These are the years when this package began to be distributed, the top date is when in Gentoo, and the bottom date is when in TDE. You should know that this ebuild is based on earlier Gentoo ebuilds. These dates should be corrected and the start year should be present. I am attaching an old ebuild from Gentoo. I wouldn’t touch this package at all, since although it’s going to be put together, it’s not working. When trying to read the disk information it just freezes. You should keep an eye on the dates, as they are added for a reason.
G-Src commented 3 months ago
Poster
Collaborator

header.txt doesn't have the beginning year so i removed the beginning year too. should i keep it?

The header.txt file has nothing to do with it at all.
These are the years when this package began to be distributed, the top date is when in Gentoo, and the bottom date is when in TDE. You should know that this ebuild is based on earlier Gentoo ebuilds. These dates should be corrected and the start year should be present. I am attaching an old ebuild from Gentoo.
I wouldn’t touch this package at all, since although it’s going to be put together, it’s not working. When trying to read the disk information it just freezes.
You should keep an eye on the dates, as they are added for a reason.

sorry, i'll fix it

> > header.txt doesn't have the beginning year so i removed the beginning year too. should i keep it? > > The header.txt file has nothing to do with it at all. > These are the years when this package began to be distributed, the top date is when in Gentoo, and the bottom date is when in TDE. You should know that this ebuild is based on earlier Gentoo ebuilds. These dates should be corrected and the start year should be present. I am attaching an old ebuild from Gentoo. > I wouldn’t touch this package at all, since although it’s going to be put together, it’s not working. When trying to read the disk information it just freezes. > You should keep an eye on the dates, as they are added for a reason. sorry, i'll fix it

Reviewers

SlavekB approved these changes 3 months ago
The pull request has been merged as eb8794e579.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/tde-packaging-gentoo#359
Loading…
There is no content yet.