Swap rpmSpecBuild() arguments for consistency with other APIs
authorPanu Matilainen <pmatilai@redhat.com>
Thu, 2 Sep 2010 11:39:11 +0000 (14:39 +0300)
committerPanu Matilainen <pmatilai@redhat.com>
Thu, 2 Sep 2010 11:39:11 +0000 (14:39 +0300)
- What we're building comes first, /how/ we're building comes second
- Update the sole caller, doesn't matter as this is a new interface...

build/build.c
build/rpmbuild.h
rpmbuild.c

index 3e957ca..fe65bc1 100644 (file)
@@ -302,7 +302,7 @@ exit:
     return rc;
 }
 
-rpmRC rpmSpecBuild(BTA_t buildArgs, rpmSpec spec)
+rpmRC rpmSpecBuild(rpmSpec spec, BTA_t buildArgs)
 {
     /* buildSpec() can recurse with different buildAmount, pass it separately */
     return buildSpec(buildArgs, spec, buildArgs->buildAmount);
index 78981a9..8a9c154 100644 (file)
@@ -87,12 +87,12 @@ rpmps rpmSpecCheckDeps(rpmts ts, rpmSpec spec);
 rpmds rpmSpecDS(rpmSpec spec, rpmTag tag);
 
 /** \ingroup rpmbuild
- * Build stages state machine driver.
- * @param buildArgs    build arguments
+ * Spec build stages state machine driver.
  * @param spec         spec file control structure
+ * @param buildArgs    build arguments
  * @return             RPMRC_OK on success
  */
-rpmRC rpmSpecBuild(BTA_t buildArgs, rpmSpec spec);
+rpmRC rpmSpecBuild(rpmSpec spec, BTA_t buildArgs);
 
 #ifdef __cplusplus
 }
index fe604ba..a8b6b80 100644 (file)
@@ -472,7 +472,7 @@ static int buildForTarget(rpmts ts, const char * arg, BTA_t ba)
        goto exit;
     }
 
-    if (rpmSpecBuild(ba, spec)) {
+    if (rpmSpecBuild(spec, ba)) {
        goto exit;
     }