This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	indygreg	D1847 rust: add TODO about lifetime of program_name variable
Closed	indygreg	D1846 rust: avoid redundant 'static lifetime
Closed	indygreg	D1845 rust: move import of PathBuf

Diff 4791

	//println!("{:?}", env);			//println!("{:?}", env);

	// Tell Python where it is installed.			// Tell Python where it is installed.
	set_python_home(&env);			set_python_home(&env);

	// Set program name. The backing memory needs to live for the duration of the			// Set program name. The backing memory needs to live for the duration of the
	// interpreter.			// interpreter.
	//			//
				// TODO consider storing this in a static or associating with lifetime of
				// the Python interpreter.
				//
	// Yes, we use the path to the Python interpreter not argv[0] here. The			// Yes, we use the path to the Python interpreter not argv[0] here. The
	// reason is because Python uses the given path to find the location of			// reason is because Python uses the given path to find the location of
	// Python files. Apparently we could define our own ``Py_GetPath()``			// Python files. Apparently we could define our own ``Py_GetPath()``
	// implementation. But this may require statically linking Python, which is			// implementation. But this may require statically linking Python, which is
	// not desirable.			// not desirable.
	let program_name = CString::new(env.python_exe.to_str().unwrap())			let program_name = CString::new(env.python_exe.to_str().unwrap())
	.unwrap()			.unwrap()
	.as_ptr();			.as_ptr();