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.

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

addpatch: notification-daemon #3193

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yuuu0724
Copy link

I have already contacted the upstream, here is the link :

dunst-project/dunst#1223

@Avimitin
Copy link
Contributor

Avimitin commented Oct 31, 2023

"File are too old" is definitely not a good summary of the error. Please consider update the upstream issue to describe what error you have met while packaging dunst project, and why the upstream should update config.guess file. You can take this issue as a great config.guess related bug report example: https://github.com/libunwind/libunwind/issues/654.

@yuuu0724
Copy link
Author

"File are too old" is definitely not a good summary of the error. Please consider update the upstream issue to describe what error you have met while packaging dunst project, and why the upstream should update config.guess file. You can take this issue as a great config.guess related bug report example: https://github.com/libunwind/libunwind/issues/654.

I have modified the script and reported back to the upstream.

@Avimitin
Copy link
Contributor

I have modified the script and reported back to the upstream.

Using "about file" as title is certainly not going to help any developer understand your request....

@yuuu0724
Copy link
Author

I have modified the script and reported back to the upstream.

Using "about file" as title is certainly not going to help any developer understand your request....

No problem, I'll make the change right away.

@yuuu0724 yuuu0724 changed the title addpkg: notification-daemon addpatch: notification-daemon Nov 1, 2023
@fwsmit
Copy link

fwsmit commented Nov 2, 2023

Hey, I'm from dunst. I don't see any config.guess file in our repo, so it seems the issue does not come from our side. Or is there something I don't understand?

@r-value
Copy link
Collaborator

r-value commented Nov 2, 2023

Hey, I'm from dunst. I don't see any config.guess file in our repo, so it seems the issue does not come from our side. Or is there something I don't understand?

I think the author of this PR might have reported the issue to the wrong place.

@XieJiSS
Copy link
Collaborator

XieJiSS commented Nov 2, 2023

@fwsmit I looked into this quickly, and it seems like a potential new contributor of this porting project has failed to find the correct upstream repo to open that issue. I think the problem is not related to dunst, but the original notification-daemon. Sorry for the noise :/

@yuuu0724
Copy link
Author

yuuu0724 commented Nov 2, 2023

Hey, I'm from dunst. I don't see any config.guess file in our repo, so it seems the issue does not come from our side. Or is there something I don't understand?

I apologize; it appears I may have been mistaken. The issue is not on your end.

@fwsmit
Copy link

fwsmit commented Nov 2, 2023

Okay, thanks for looking into it. I'll close the issue on Dunst's end

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.

5 participants