Mailinglist Archive: opensuse-buildservice (233 mails)

< Previous Next >
Re: [opensuse-buildservice] Patches to easy Stand Alone OBS install.
  • From: Andreas Bauer <abauer@xxxxxxx>
  • Date: Tue, 15 May 2007 17:20:35 +0200
  • Message-id: <200705151720.35622.abauer@xxxxxxx>
On Saturday, 12. May 2007 01:10, David B. Anderson wrote:
> Hi,
>
> Here are some patches to make setting up a stand alone OBS install easier.
> They mostly remove some hard code URLs, allowing the use of the
> environment configuration files.

Cool, thanks.

A few comments:

+  def link_to_repo(message='', repo='')
+    link_to "Build Service Software #{message}", URL_SW_REPO_HOST 
+ "/#{repo}"
+  end

I'll remove the "Build Service Software" prefix, as this makes it very 
difficult to see what the final displayed text is.

-    number_to_human_size(bytes, 2)
+    hs = number_to_human_size(bytes.to_i)
+    if hs == nil
+      hs = bytes
+    end
+    hs

I don't see the need for that. number_to_human_size can only return nil if the 
argument is nil. So your check effectively does nothing, because if hs == 
nil, bytes == nil. Additionally, the argument can never be nil as to_i, if 
defined, is guaranteed to always return an integer.

I'll check in the fixes for the hardcoded URLs if no one else has objections.

> Also included is a patch for the obs-server.spec file to make it work
> with a tarball made from current development tree.
>
> All of the patches are against r1638.

-- 
Andreas Bauer - Novell - SUSE Internal Tools
---------------------------------------------------------------------
To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: opensuse-buildservice+help@xxxxxxxxxxxx

< Previous Next >
Follow Ups
References