[Bug 1180723] Review Request: clufter - Tool for transforming/analyzing cluster configuration formats

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

 



https://bugzilla.redhat.com/show_bug.cgi?id=1180723



--- Comment #15 from Mamoru TASAKA <mtasaka@xxxxxxxxxxxxxxx> ---
(In reply to Jan Pokorný from comment #14)
> Do I understand it correctly that another suggestion is to use "%version"
> instead of "%{version}"?

- Well, usually macro is wrapped with bracket (this is preferable to
  avoid some confusion), however there are some exceptions (like
  parameterized macros) where macro cannot be wrapped.

> > %post
> 
> It was intended to allow for dynamic regeneration of Bash completion file
> upon installing/removing plugins (e.g., clufter-lib-pcs that offers a new
> command, which should be reflected in the completion file).  Currently
> this hasn't been tested, I have to look at that.

Well, as I said now bash completion files are to be installed under
%_datadir, which means that this file is not expected to be modified.
If you want to modify, it must create symlink which points to some files
under %_sysconfdir (usually under %_sysconfdir/%name), and modifying file
must be done for files under %_sysconfdir.

> Yes, I learnt this from some authoritative docs, and four characters
> overhead is negligible enough to put up with that, IMHO.

- Well, (while I no longer remember when the decision was made), there
  is already a consensus that "|| :" after %check must be removed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]