[Pkg-mc-devel] Midnight Comander 4.8.1 [RFS]

[Andreas Tille]

Hi Dmitry,

thanks for all your work on mc.  The changelog looks really impressive.

On Sat, Feb 25, 2012 at 04:45:42AM +1100, Dmitry Smirnov wrote:
> Hi Yury,
> 
> On Sat, 25 Feb 2012 03:41:49 Yury V. Zaytsev wrote:
> > 
> > And yeah, another idea: also maybe you can replace raster icons with the
> > render from svg and change all names to midnight-commander for
> > consistency (mc is too short to avoid conflicts, I'm afraid)...
>  
> I could, but is it really necessary to do everything in one go? 

IMHO it is better to get that real lot of fixes out as fast as possible
and do cosmetics later.

> If I stay at least for some time I might have a chance to prepare another 
> upload. :)

Sounds promising.
 
> There are so many different workflows and some people can use repository for 
> the same purpose or wiki or even the bugtracker itself with tagging etc.
> I'm always happy to learn how to do the work better.
 
IMHO the bugs.txt file makes sense when working offline.  I think
maintaining this file is not mandatory for an upload because while its
helpful for other developers as well it is not visible to users who just
want to recive the new package (urgently). 

The only thing which might be interesting to do before an upload whould
IMHO be if Maarten would check the according Launchpad bugs and add
these to the changelog.  However, this should also be no real reason for
a delay because I guess these could also be closed manually.
 
> > But ok, let's keep it for later, right now the top priority is to bring
> > the package up to date and push it out.
> 
> That would be nice.

I have some remarks to your work.  When adding a remark about the
changed VCS the SVN commit hook sended a mail to *this* list
(pkg-mc-devel).  My recent change in the new repository where I bumped
the Standards-Version to the recently released policy version went via
Commit hook to the commit list (pkg-mc-commits).  I think the new
behaviour is perfectly OK but want to mention here that the general
behaviour has changed.  Please everybody who is involved notice this.

Another think I noticed is that my long used alias 

   alias mc='. /usr/share/mc/bin/mc-wrapper.sh'

is not working any more.  It either needs to be replaced by

   alias mc='. /usr/lib/mc/mc-wrapper.sh'

and this should be mentioned in a NEWS.debian entry or we need to find
some in-package means to keep the old behaviour working (either via a
symlink to the old location or using the alias system).  I have no
opinion what might be the best option, but we should probably not leave
this issue unhandled.

Kind regards

      Andreas.

-- 
http://fam-tille.de