[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1041824: marked as done (src:volume-el: disable d/watch and sync to latest head version)

Your message dated Wed, 26 Jul 2023 08:56:43 +0100
with message-id <87wmynds1w.fsf@melete.silentflame.com>
and subject line Re: Bug#1041824: src:volume-el: Enable merge request on salsa
has caused the Debian Bug report #1041824,
regarding src:volume-el: disable d/watch and sync to latest head version
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
1041824: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1041824
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Source: volume-el
Severity: wishlist
X-Debbugs-Cc: none, Xiyue Deng <manphiz@gmail.com>

Dear maintainers,

I have been trying to fix uscan error of Emacs addon packages.  When
working on volume-el, I found that the repo on salsa didn't accept merge
requests while most other packages did.  If it can open up merge request
access it would be great and I have some pending d/watch fixes.  Thanks
in advance!


-- System Information:
Debian Release: 12.1
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable-security'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 6.1.0-10-amd64 (SMP w/16 CPU threads; PREEMPT)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

--- End Message ---
--- Begin Message --- 
control: tag -1 + wontfix

Hello,

Thank you for these submissions.

On Sun 23 Jul 2023 at 08:48pm -07, Manphiz wrote:

> * Sync to latest head version, which basically just incorporated Sean's
>   patch upstream so that we don't need to host the patch anymore.

Patches don't really work for merging new upstream releases -- in this
case, pushing a branch somewhere and inviting me to merge it is better.
For proposed new changes rather than merges, I do indeed prefer patches,
so thank you for preparing those.

In this case, I would prefer not to merge the new upstream, because it
doesn't actually change the package.  It's not a problem carrying the
patch, and I think it's preferable.

As for your first patch, I don't follow -- I think I already fixed that
a while ago?  Indeed, the patch does not apply to our team's repository.

I would prefer not to apply the final patch until the QA team come and
tell us that we should disable watch files like that.

-- 
Sean Whitton

Attachment: signature.asc
Description: PGP signature


--- End Message ---
Reply to: