[Bug 979677] Review Request: perl-CPANPLUS-Dist-Fedora - CPANPLUS backend to build Fedora/RedHat RPMs

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

 



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

--- Comment #13 from Petr Šabata <psabata@xxxxxxxxxx> ---
(In reply to Parag AN(पराग) from comment #10)
> (In reply to Petr Šabata from comment #9)
> > How about the first paragraph?
> > 
> > "Having proper build requirements saves the time of all developers and
> > testers as well as autobuild systems because they will not need to search
> > for missing build requirements manually. It is also a safety feature that
> > prevents builds with that would not otherwise fail, but would be missing
> > crucial features."
> 
> This stands correct if mock build will fail and we add that missing BR: and
> build will be successful.
> 
> > What if I change something in your dependency chain and your build breaks?
> 
> In that case we can add manually required missing BR:
> 
> > Do you prefer to wait until you get a FTBFS to listing things your package
> > actually uses?
> 
> We can work on this when there will be FTBFS reported.

Why do you refuse to prevent the FTBFS in the first place?

> > I still fail to see how this is perl specific.
> 
> I will re-iterate mock is not failing so its with perl guidelines to specify
> that though a single dependent perl module BR: will bring perl package in
> buildroot, we need to write it BR: perl in spec files.
> 
> > This attitude causes so much slowdown every rebuild it's ridiculous.
> 
> Again, Its something that perl guidelines are missing....

As the way how we treat dependencies doesn't affect just Perl but other package
group rebuilds too (including Fedora mass rebuilds), I'd prefer rewording the
general Guidelines to be more enforcing on this matter.  Treating each package
group differently and possible duplication of Guidelines isn't the best option.

(Still, having this in Perl SIG guidelines would be better than nothing...)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=oLto0J8qh7&a=cc_unsubscribe
_______________________________________________
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]