[Bug 3152] Review request: dropbox-repo - 3rd-party repo package for Dropbox client

RPM Fusion Bugzilla noreply at rpmfusion.org
Thu Feb 6 15:50:14 CET 2014


--- Comment #45 from rc040203 at freenet.de 2014-02-06 15:50:14 CET ---
(In reply to comment #43)
> (In reply to comment #42)
> > (In reply to comment #41)
> > > No, %global will not work, the use here requires lazy evaluation.
> > Wrong, you are using %name before it's defined.
> Yes, that's the current usage and the reason %define is used instead of
> %global. It's a deliberate decision form my side.

Why? IMO, what you are doing is bad coding.

> > Are the reasons behind the requirement to increment the NVR each time you
> > change something really so difficult to comprehed?
> Hey, is this personal?
Not yet, but to be honest, you are gradually provoking it.

May be you don't notice that not incrementing the NVR unneccessarily
complicates reviews.

> > Finally, how do you plan to handle the case of 3rd parties adding
> > /etc/yum.repos.d/*repo files themselves?
> > 
> > Note: Any remote repository, at any time can do so, which at minium will break
> > a user's installation.
> For the case there already is a repo file installed when installing
> dropbox-repo refer to 
> https://lists.rpmfusion.org/pipermail/rpmfusion-developers/2014-February/016552.html
>  (near bottom).
> If one tries to install a package with a repo file after dropbox-repo is
> insrtalled, there will be a conflict. Isn't this as it should?
In Fedora, conflicts are not supposed to happen.

> User can always edit the repo file. After all, it's a configuration file.
A file marked %config will not conflict, but are you sure an upstream added
/etc/yum.repos.d/*repo be marked %config?

As I repeatly said, it's all out of your control, so you must take maximum
precautions for your package to work smoothly. ATM, I don't have any idea how
to achieve this.

Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.

More information about the rpmfusion-developers mailing list