fix: don't pass --nodeps to initial build step

This fixes packages that require their dependencies in `pkgver` or
`prepare` failing to build.

.woodpecker/.docker.yml

@@ -11,7 +11,9 @@ pipeline:
       - 'docker_password'
     settings:
       repo: 'chewingbever/vieter'
-      tag: 'dev'
+      tags:
+        - 'dev'
+        - ${CI_COMMIT_SHA}
       platforms: [ 'linux/arm64/v8', 'linux/amd64' ]
       build_args_from_env:
         - 'CI_COMMIT_SHA'

docs/content/builder.md

@@ -15,7 +15,7 @@ repositories. After the image has been created, each repository returned by
 previously created image as a base. Each container goes through the following steps:
 
 1. The repository is cloned
-2. `makepkg --nobuild --nodeps` is ran to update the `pkgver` variable inside
+2. `makepkg --nobuild` is ran to update the `pkgver` variable inside
    the `PKGBUILD` file
 3. A HEAD request is sent to the Vieter server to check whether the specific
    version of the package is already present. If it is, the container exits.

src/build/build.v

@@ -91,7 +91,7 @@ pub fn build_repo(address string, api_key string, base_image_id string, repo &db
 	commands := [
 		'git clone --single-branch --depth 1 --branch $repo.branch $repo.url repo',
 		'cd repo',
-		'makepkg --nobuild --nodeps',
+		'makepkg --nobuild',
 		'source PKGBUILD',
 		// The build container checks whether the package is already
 		// present on the server
@@ -118,11 +118,7 @@ pub fn build_repo(address string, api_key string, base_image_id string, repo &db
 	mut data := docker.inspect_container(id) ?
 
 	// This loop waits until the container has stopped, so we can remove it after
-	for {
-		if !data.state.running {
-			break
-		}
-
+	for data.state.running {
 		time.sleep(1 * time.second)
 
 		data = docker.inspect_container(id) ?

src/docker/containers.v

@@ -67,13 +67,6 @@ pub mut:
 	end_time   time.Time [skip]
 }
 
-fn docker_timestamp_to_time(s string) ?time.Time {
-	parts := s.split('.')
-	clipped := parts[0] + '.' + parts[1][..3]
-
-	return time.parse_rfc3339(clipped)
-}
-
 // inspect_container returns the result of inspecting a container with a given
 // ID.
 pub fn inspect_container(id string) ?ContainerInspect {
@@ -85,10 +78,10 @@ pub fn inspect_container(id string) ?ContainerInspect {
 
 	mut data := json.decode(ContainerInspect, res.text) ?
 
-	data.state.start_time = docker_timestamp_to_time(data.state.start_time_str) ?
+	data.state.start_time = time.parse_rfc3339(data.state.start_time_str) ?
 
 	if data.state.status == 'exited' {
-		data.state.end_time = docker_timestamp_to_time(data.state.end_time_str) ?
+		data.state.end_time = time.parse_rfc3339(data.state.end_time_str) ?
 	}
 
 	return data
@@ -101,6 +94,8 @@ pub fn remove_container(id string) ?bool {
 	return res.status_code == 204
 }
 
+// get_container_logs retrieves the logs for a Docker container, both stdout &
+// stderr.
 pub fn get_container_logs(id string) ?string {
 	res := request('GET', urllib.parse('/v1.41/containers/$id/logs?stdout=true&stderr=true') ?) ?
 	mut res_bytes := res.text.bytes()