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

RPM Fusion Bugzilla noreply at rpmfusion.org
Thu Feb 6 15:13:57 CET 2014


--- Comment #43 from Alec Leamas <leamas.alec at gmail.com> 2014-02-06 15:13:57 CET ---
(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.

> Move it down after Name: and %global will start working.
> Alternatively, you try to derive %reponame from %name or another common
> %global.
> > I have added
> > a justification in the spec. Updated in-place, same links.
True. It's just that all of these options IMHO would give a more cluttered
spec. So if the reviewer is OK with it, I'd prefer to have it as is although I
of course could modify along these lines. No big deal, at least not for me :)

> Are the reasons behind the requirement to increment the NVR each time you
> change something really so difficult to comprehed?
Hey, is this personal?

> 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 
 (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?

User can always edit the repo file. After all, it's a configuration file.

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