From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id C110242C3E; Tue, 6 Jun 2023 12:19:04 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6B34E40697; Tue, 6 Jun 2023 12:19:04 +0200 (CEST) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by mails.dpdk.org (Postfix) with ESMTP id C9C1140223 for ; Tue, 6 Jun 2023 12:19:01 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1686046742; x=1717582742; h=date:from:to:cc:subject:message-id:references: content-transfer-encoding:in-reply-to:mime-version; bh=bJtbYyiU9pcseLCIvMWmmAUj3cXk95dWpk1lhVne3Hw=; b=kuIVW7+5KQsrZeQ6n5nJ5JxljJrHzkksPy/zSYhX2KzHlSwCkzRHVP/Q NKbYPSNckkXtPQvRFBeS1j6xhKsD5Q4f8SfKcPALHUwHByRAUazFS0hwj MJsF2Djz6pN7g8UvEj6oTIji7Cn4ZN1vqnJ09wj4Mzb54ttFxIrnDGENu b0v047QIEliWW0BSHX4IsOnnHEIbsuanSzkhFMS2BplZwiJF6SOuDmnzT gXRTcynyu/51YWSfF+KFPA8GHB4LxbQp4W6lBvzCvQ3+MJIbC8WNwa1xX jdMOBCZGF7rwhJHzV1ZV8ZfqVcbsf/p4qHNOc6SCZjsn4lDJEEcD1Eu1h Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10732"; a="443004122" X-IronPort-AV: E=Sophos;i="6.00,221,1681196400"; d="scan'208";a="443004122" Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Jun 2023 03:19:00 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10732"; a="709032834" X-IronPort-AV: E=Sophos;i="6.00,221,1681196400"; d="scan'208";a="709032834" Received: from fmsmsx603.amr.corp.intel.com ([10.18.126.83]) by orsmga002.jf.intel.com with ESMTP; 06 Jun 2023 03:18:59 -0700 Received: from fmsmsx612.amr.corp.intel.com (10.18.126.92) by fmsmsx603.amr.corp.intel.com (10.18.126.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.23; Tue, 6 Jun 2023 03:18:59 -0700 Received: from fmsmsx603.amr.corp.intel.com (10.18.126.83) by fmsmsx612.amr.corp.intel.com (10.18.126.92) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.23; Tue, 6 Jun 2023 03:18:59 -0700 Received: from FMSEDG603.ED.cps.intel.com (10.1.192.133) by fmsmsx603.amr.corp.intel.com (10.18.126.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.23 via Frontend Transport; Tue, 6 Jun 2023 03:18:59 -0700 Received: from NAM02-DM3-obe.outbound.protection.outlook.com (104.47.56.47) by edgegateway.intel.com (192.55.55.68) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.23; Tue, 6 Jun 2023 03:18:58 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=V174B3FE3plpnelMdyfCri8Q8a8KCTTgEs+jQxSICfICxGjipmMNCzUd160zlwU/IPP7JQ8cPh869/Jatq/vJPcpkehKg1La/Pa+AsQ1nxtMq/QzHXX5lcw2FAVWnfn+uuL7Lw/TvtkrjyF5+v/k6N8UvQLWIV0WCulEW7Xsk3pMZHmQHlaFZqhBf1N9gI9aWXy1RomO3kkI3uNSXjEQzPAUxDaiqKGSdSeBH3f16rCQXayzBeY3JWRy8gxoSG/gFOZRrpOgWXKQ68ZW8Lid3Kwdgicnp5ujU01ROCspQKY1IAVwSKCdWWwVtfH53DztjfdV9g979OgOLPX7h+JvRA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=RjuRdKFGI7RZVmE75/rEjD2h7W/XM+Uqqcyuybx3zqw=; b=Aclm58LXGSFavfRiO7tqI8OAgz2CZGDGMMUxjbg3HrH1JIwbaj9zMJlp9tivv7X3pJiWTtCws9ZQPxBNhDMOivZrdu3oyxY/MsTDXz3fyHfqViiljBnnXDpqAyfvEdRJz8zKQG3IAMlk5uR4Tx1lucOXU7q8pDIXDMT2FX3/9RZPjOymmSZHYWEqn0ohpPBFlLJTVavRrOHPCWMvfXrIVnEi1HdrdVqeJtZFKylJu78Qr/+Nx57o6eL9HzSfC7g0PgKk/oWoc0knDHabhEqiRvJH1ktNQtk2UlVuatekaLJ2obvnvfUxhNLTdYMQh+9hy5f00CvVZzOPqZZMC2zAxw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; Received: from DS0PR11MB7309.namprd11.prod.outlook.com (2603:10b6:8:13e::17) by PH0PR11MB5013.namprd11.prod.outlook.com (2603:10b6:510:30::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6455.33; Tue, 6 Jun 2023 10:18:57 +0000 Received: from DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::b8f3:958:d2c5:2232]) by DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::b8f3:958:d2c5:2232%7]) with mapi id 15.20.6455.030; Tue, 6 Jun 2023 10:18:56 +0000 Date: Tue, 6 Jun 2023 11:18:50 +0100 From: Bruce Richardson To: Jerin Jacob CC: Subject: Re: [PATCH] doc: build manpages as well as html output Message-ID: References: <20230601153801.118616-1-bruce.richardson@intel.com> Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-ClientProxiedBy: LO4P123CA0673.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:351::8) To DS0PR11MB7309.namprd11.prod.outlook.com (2603:10b6:8:13e::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS0PR11MB7309:EE_|PH0PR11MB5013:EE_ X-MS-Office365-Filtering-Correlation-Id: 7f770c9a-6456-4cda-3a47-08db66776fca X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: hnsBBL5NPp1sE0rQjVl4c9kdIwkwmAn7m4FctRK+tBXclVYOogcFH1ejRWpizMu+qpSfr7HTUtYrKTckbPjB9uQIRphxPQRZK8jg5PPiOfqpHlmKtQCIK8HhXyA91jhnVCG2RxRoFNcluZ2rghKUxoJqYNfRYjEJw11wSRdeZf4OwbeVM7YU5zVPlTdXxTHMzhbPK7DRFQSwwnkj6Bs5QEOp03tkrbNE4jasRN/s8CTKxpAArpPKcAtMkfeD/786mZJ5G0yNzGo0vXUsTdMNQXw2vb98kGWrh1FsiIlKB3/AD3zewVfDX9RBsUxXbr0mRjiknTuyxC9kCZ36icgDwXf20vHNHiQX2hCbmu9Zbg2does8G7NigUjmgXdAyFFs0C/XtmyI8Nh1DWGtGINuXykWqei/7cNoIDq7TTDaA6c9kfkK651gko8bZwsgHL2vQ2S7B9f7p2ejpFN0s5gTklmq/oAeUx1ZfFolv2wiJf96IeGTwDUg+Z5eirMCF2588X2EkUiM0nlx6HWJQwn4qhsWn8czJK+2bfzB001PBAyTjGsjKYQabejXF4FSMgIP X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DS0PR11MB7309.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(346002)(366004)(136003)(39860400002)(396003)(376002)(451199021)(186003)(82960400001)(478600001)(6916009)(8676002)(8936002)(4326008)(41300700001)(66946007)(38100700002)(316002)(66556008)(66476007)(83380400001)(6486002)(6666004)(26005)(6512007)(6506007)(53546011)(86362001)(44832011)(5660300002)(2906002); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?L1NmTzlWOWpYbFVBazUrZWxCK2djZ0lZYlAwZkcrSnpIL3YyeGVLMm9VMWpm?= =?utf-8?B?c2trelpnaUNnZFNCQWdKWGJJWTZUUEQ3QUNHTkNTSS9BcjdrV0hxSE1BeXE5?= =?utf-8?B?Y281VEdpK1RBZVQvWWlmVzZ2dzFCNlcrd0R3eG11UXkxMGROc2kxUGY3ZzNl?= =?utf-8?B?Z1V6UGhheVkwbXdsTEord0xjd3pqSEVqb0JqbENnT1NzektLS25kUGo4Qzg2?= =?utf-8?B?K1FPdW43M3dIZCs4Wk5TV2pyK0lGT1MvVjRMK3Z3VTQ2VzdIUVNYa21hdElj?= =?utf-8?B?Yy9yRlQyUFFad3NSbXFGQU5PSHh5anFqakN3SXlUUkxKZ0VkL25JcFh6czZT?= =?utf-8?B?aWg1bENaN2VXcm1xQ1hhUENucFlTemYzZVAxVENPZHFzWjA5L0FqejR6T0Zp?= =?utf-8?B?MVFIRzJDSitvMGxPaXJlR0xwWmRHS1AzMWxBU2dXU0FTVFNRdGdTRWlVcTdX?= =?utf-8?B?VCtrZlFmVDBnT3ZiZXJIa2dHNFVPSUt1dUxtNkFkZGRSRkkvMDA1SVEwT3p0?= =?utf-8?B?T1VNRHVYSmtvWERhRDVvN21hNjF4ZnVkT0d5RkhRZThwdjdKUC8vUExJZE9k?= =?utf-8?B?M2tlVFQxS3Q1OGI0dlFNSU9oVFRsY2xabEl0WUpNbkhJR2JYTENsUzFHbHBm?= =?utf-8?B?L1J6amNwaldjdlZwdzRyS3BYZytoOVk1eXBGOWo4V2Y4cjVPU3lCdjdZWEFu?= =?utf-8?B?SlFrbVBJN1lGYjNKRXhmMmN0ZU5iQzRIQW1jWXFaejNjd1Azby90MS80cUw2?= =?utf-8?B?OWViekpVUzcvVFFhQTl1WWxYT2loUXNXMEg0RGpNQWxiU1FMYkxaM3R2cFdI?= =?utf-8?B?RVhpV0FKajl2OVJuUjRvZHY0MjRuOFRMTHY2NDVuTWY5ZXh0VU5vZGgwdHJF?= =?utf-8?B?YkZvc1lKb2EzZ3VKdlg1U2RaZTNTb1RoT0lJOGtWYUpBN2hjc1RXT3BMNUZk?= =?utf-8?B?V2RUVjFVcVlFRGZUbHh6bjFncitBcmc0a2dlYVIvS2FkcnlubUZoZGt3NWdo?= =?utf-8?B?SXhHRjVQK0kwTTZxUHpSSXFMVVR1RzNJeDA1YjV3WVR1VU9MZDduRWIwTlF2?= =?utf-8?B?WVpIdVZGZFoxMmhQNXg3OHFNbG5iZUViTXVnQmVEblBXN1NwVzdtOVMyeENR?= =?utf-8?B?djAzbGorTnZudVRPaUFXSmFwTVJYUGR1a2ltZ0xnU0ZqdlVPOUloTisxaFpm?= =?utf-8?B?ek5PNGlDUjBVMG51Z3NtUGNnVjk4UjhreWtNNWE4OXlNREk3MDQ5aGZLQ01u?= =?utf-8?B?SGNpbWcxNm5kNWtKQ2dFMXpVdGppamI4RlM1VGs0TjZrNDh1aWlSU1QvbStD?= =?utf-8?B?T1RUNWtiYTRLZERHdUt2d2FSUVJwazF0Nm1vWklRNCtyZ2UrVkhVRlBYdWVy?= =?utf-8?B?U1M5YWhuV0dJTzJIWGI4cVczVFdNVDFWWmc5T09BLy9URUQzVnRPdjQ3ajlu?= =?utf-8?B?cFZ3Nmk3QW1GaDdOZHc2OGNMTndXQnlIejJyZ2ZOTzMrcHNZQXVoOTNXa1Nn?= =?utf-8?B?LzByeTlKdVdmSjdIRUJTNXdCZHlZREE1NW1Xd3gyUFlTOVdJd3JtckxYaEVB?= =?utf-8?B?M1Z0dVBYK3JHczNIc1hLN2p5YnFuUEpDSGFEYnVDNUVGMWZsWGlJREFBc1hT?= =?utf-8?B?bmovQkVJTSt3aWtPa1NtdXVRM2FIVmpZQWk5SjRZS1hCL1F0SGpTc0lPV2JZ?= =?utf-8?B?UXhkTUJ3UU5kWkpicHZ6MWVncVQ0ak5Bd1IyT203NmFiMTZZZzZ5bFpXajNR?= =?utf-8?B?TkhiR1NMeFQyRERobHN4eFFRc3J5dE0vMnFOSGhBa3dZMFNGZG9nUkxpcWls?= =?utf-8?B?dGFwSWJ5WHNKRElPdGF6YThFdHlRMDFBcXcyaFBSRG9FNHhxaS9lTGJzVmNW?= =?utf-8?B?eGlaVXhtUXJ6bHNhTy9rcGpFdWhkcTg1RUtjcy91d3k1SnZHUGxyU3h4dXlh?= =?utf-8?B?RmhHV1VyOGE1emlFeGhqdHI3cDNPSVRNb0FNYlBSNTBXV0NCdmZ0dVpDUDZF?= =?utf-8?B?b2ZlVmlLcjZKbTF6cmlmMzhKOXdMcVBjL3Y4eVRnZkNVVzR6N1B5Mk5iUWdE?= =?utf-8?B?NmNUbE0yaE9RT0N3K2dEYVNHSHJoMEs5WTlYUDdiZm9CV0JpenJrTDlKbnNt?= =?utf-8?B?alFCRmE0dmxrRDBMTWM2SlYrUVcrRkE2RGVCVlA0cWdQMDRZeFpVaEU5RE15?= =?utf-8?B?b0E9PQ==?= X-MS-Exchange-CrossTenant-Network-Message-Id: 7f770c9a-6456-4cda-3a47-08db66776fca X-MS-Exchange-CrossTenant-AuthSource: DS0PR11MB7309.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jun 2023 10:18:56.8182 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: plEiLJOW+c63BJDPo02mGGjKZL5Ohyfk4U+la/uvGYADnfByR0a4cv3MNdJHPAY+wUFzegn65XKlOEWSzdKXe5wVHRDySvwkM6ZN/XNnTQs= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5013 X-OriginatorOrg: intel.com X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On Tue, Jun 06, 2023 at 03:16:27PM +0530, Jerin Jacob wrote: > On Tue, Jun 6, 2023 at 2:49 PM Bruce Richardson > wrote: > > > > On Mon, Jun 05, 2023 at 10:50:48AM +0530, Jerin Jacob wrote: > > > On Thu, Jun 1, 2023 at 9:08 PM Bruce Richardson > > > wrote: > > > > > > > > Doxygen can produce manpage output as well as html output for the DPDK > > > > APIs. However, we need to do this as a separate task as the manpage > > > > output needs to be placed in a different location post-install to the > > > > html output (/usr/local/share/man vs /usr/local/share/doc/). > > > > > > > > Changes required are: > > > > * Add configurable options for manpage output and html output to the > > > > doxygen config template. (Remove option for html output path as it's > > > > always "html") > > > > * Modify API meson.build file to configure two separate doxygen config > > > > files, for HTML and manpages respectively. > > > > * Change doxygen wrapper script to have separate output log files for > > > > the manpage and HTML jobs, to avoid conflicts > > > > * Add "custom_targets" to meson.build file to build the HTML pages and > > > > the manpages, with individual install locations for each. > > > > * Where supported by meson version, call "mandb" post-install to update > > > > the man database to ensure the new manpages can be found. > > > > > > > > Signed-off-by: Bruce Richardson > > > > > > > + > > > > +mandb = find_program('mandb', required: false) > > > > +if mandb.found() and get_option('enable_docs') and meson.version().version_compare('>=0.55.0') > > > > + meson.add_install_script(mandb) > > > > > > It does not look like just executing mandb it is adding these man > > > pages to database > > > > > > log: > > > Running custom install script '/usr/bin/mandb' > > > Purging old database entries in /home/jerin/.local/man... > > > Processing manual pages under /home/jerin/.local/man... > > > Checking for stray cats under /home/jerin/.local/man... > > > Processing manual pages under /home/jerin/.local/man/cat1... > > > Purging old database entries in /home/jerin/.local/share/man... > > > Processing manual pages under /home/jerin/.local/share/man... > > > Checking for stray cats under /home/jerin/.local/share/man... > > > Processing manual pages under /home/jerin/.local/share/man/cat1... > > > 0 man subdirectories contained newer manual pages. > > > 0 manual pages were added. > > > 0 stray cats were added. > > > 0 old database entries were purged. > > > > > > [main][dpdk.org] $ man rte_flow_create > > > No manual entry for rte_flow_create > > > > > > # Following works by providing the path i.e man pages created properly > > > only db update is missing > > > man --manpath=/tmp/i/usr/local/share/man/ rte_flow_create > > > > > Yes, that is correct. > > > > If you install to a non-standard location, then yes you need to update > > manpath yourself. However, in case you install to a "standard" location, > > then running mandb will update the database for you. I believe this is the > > behaviour we should have. I view it as the same as installing binaries in a > > standard vs non-standard path - if the binaries are placed in a standard > > location then they are found automatically, but if installed in a custom > > location, then the user is responsible for ensuring all paths are correct. > > OK. Then I think, we can move "meson.add_install_script(mandb)" > invocation under !DESTDIR not provided. > As if DESTDIR is provided then there is no use for mandb. > That would mean adding a new build script to run mandb, since: * meson doesn't directly support getting environment variables * DESTDIR may only be set at install time rather than at configuration time, so we'll always need to set up the build script Furthermore, I'd point out that DESTDIR is not going to avoid unnecessary runs of mandb, since using a special "prefix" setting would also cause the manpages to not be found. Overall, I'd view the complexity of avoiding the mandb call not to be worth it, since calling mandb is largely harmless - bar the time taken to run it. However, if you feel strongly it should be skipped when installing with DESTDIR, I'll spin a v2 with the wrapper script added. Let me know what you think? /Bruce