Fix traditional builder jobname handling (#1693)

If job name is specified e.g. via TEX directives, before this commit ...

1. latexmk failed to build the document with error 10.
2. logfile and created pdf documents were not found.

Author: deathaxe

latextools/make_pdf.py

@@ -161,7 +161,7 @@ def worker(self, activity_indicator):
             cmd_coroutine.close()
 
         try:
-            log_filename = f"{self.caller.builder.base_name}.log"
+            log_filename = f"{self.caller.builder.job_name or self.caller.builder.base_name}.log"
             if self.caller.builder.aux_directory_full:
                 log_file = os.path.join(self.caller.builder.aux_directory_full, log_filename)
                 if not os.path.exists(log_file):

plugins/builder/pdf_builder.py

@@ -335,7 +335,7 @@ def copy_assets_to_output(self) -> None:
         """
         if self.aux_directory_full != self.output_directory_full:
             for ext in (".synctex.gz", ".pdf"):
-                asset_name = self.base_name + ext
+                asset_name = (self.job_name or self.base_name) + ext
 
                 dst_file = os.path.join(self.output_directory_full, asset_name)
                 try:

plugins/builder/traditional_builder.py

@@ -108,22 +108,22 @@ def commands(self) -> CommandGenerator:
                 # if documents are opened and locked by viewer on Windows.
                 cmd.append(f"-output-directory={self.aux_directory_full}")
 
-            if self.job_name != self.base_name:
-                cmd.append(f'-jobname="{self.job_name}"')
-
             cmd += map(lambda o: f"-latexoption={o}", self.options)
 
+            if self.job_name != self.base_name:
+                cmd.append(f'-jobname={self.job_name}')
+
         elif texify:
             # no need to output messages, if they are not consumed
             if logger.getEffectiveLevel() == DEBUG or self.display_log:
                 cmd.append("--verbose")
             else:
                 cmd.append("--quiet")
 
-            if self.job_name != self.base_name:
-                cmd.append(f'--job-name="{self.job_name}"')
-
             cmd += map(lambda o: f'--tex-option="{o}"', self.options)
 
+            if self.job_name != self.base_name:
+                cmd.append(f'--job-name={self.job_name}')
+
         # texify wants the .tex extension; latexmk doesn't care either way
         yield (cmd + [self.tex_name], f"running {cmd[0]}...")