Skip to content

Fix Vivado missing IP repository warning on clean builds#118

Open
taraxacum45e9a wants to merge 1 commit into
Xilinx:devfrom
taraxacum45e9a:pr-iprepo
Open

Fix Vivado missing IP repository warning on clean builds#118
taraxacum45e9a wants to merge 1 commit into
Xilinx:devfrom
taraxacum45e9a:pr-iprepo

Conversation

@taraxacum45e9a

@taraxacum45e9a taraxacum45e9a commented Jun 14, 2026

Copy link
Copy Markdown

config.ip_repository defaults to a subdirectory inside build_dir.

def ip_repository(self) -> Path:
return self.build_dir / "iprepo"

Because build_dir is wiped and recreated on every build, ip_repository may not exist yet when create_build_project is called.

if self._build_dir.is_dir():
shutil.rmtree(self._build_dir)
self._build_dir.mkdir(parents=True)

Previously it was unconditionally appended to the Vivado command line, which could cause the tool to warn on a clean build.

IP REPOS:       <redacted>/SLASH/linker/install.prj/iprepo

INFO: Creating new project 'slash' in '<redacted>/SLASH/linker/install.prj' ...
INFO: [IP_Flow 19-234] Refreshing IP repositories
WARNING: [IP_Flow 19-2248] Failed to load user IP repository '<redacted>/SLASH/linker/install.prj/iprepo'; Can't find the specified path.
If this directory should no longer be in your list of user repositories, go to the IP Settings dialog and remove it.

The fix checks config.ip_repository.exists() before appending, so the argument is only passed when the directory is actually present.

@taraxacum45e9a taraxacum45e9a changed the title Clean up subprocess cwd handling and fix duplicate IP repo in Tcl script Clean up subprocess cwd handling and fix missing IP repo in Tcl script Jun 14, 2026
config.ip_repository defaults to a subdirectory inside build_dir.
Since build_dir is recreated fresh on every build, ip_repository
may not exist yet.

Check for its existence before appending it to the command to prevent
errors.

Signed-off-by: Shen Jiamin <shen_jiamin@comp.nus.edu.sg>
@taraxacum45e9a taraxacum45e9a changed the title Clean up subprocess cwd handling and fix missing IP repo in Tcl script Fix Vivado missing IP repository warning on clean builds Jun 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant