Opened 10 years ago

Closed 4 years ago

#44763 closed defect (worksforme)

gnutls @3.3.7 +staging into destroot error

Reported by: LeonardEighteen@… Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.3.1
Keywords: Cc: Schamschula (Marius Schamschula), chrstphrchvz (Christopher Chavez)
Port: gnutls

Description

Sorry for the previous post and thanks for your patience. My system is OS X 10.9.4, and when I tried to upgrade to gnutls @3.3.7, I encountered error while staging gnutls into destroot. Please view my main.log in the attachment.

Attachments (1)

main.log (686.1 KB) - added by LeonardEighteen@… 10 years ago.

Download all attachments as: .zip

Change History (15)

Changed 10 years ago by LeonardEighteen@…

Attachment: main.log added

comment:1 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)

Keywords: staging into destroot error removed
Priority: HighNormal

Looks like a texinfo issue. Do you have the texinfo port installed? If not, try installing it; gnutls may need a newer texinfo than the OS provides. On the other hand, if you already have texinfo installed, try deactivating it; we've found many projects aren't yet compatible with texinfo 5.

comment:2 Changed 10 years ago by neverpanic (Clemens Lang)

It builds fine for me without the texinfo port, and gnutls doesn't have a texinfo dependency, so I assume this is another incompatibility that should be reported to upstream to fix. Meanwhile, try cleaning and running sudo port -t upgrade gnutls, which should hide texinfo and allow the build to finish.

comment:3 in reply to:  1 Changed 10 years ago by LeonardEighteen@…

Replying to ryandesign@…:

Looks like a texinfo issue. Do you have the texinfo port installed? If not, try installing it; gnutls may need a newer texinfo than the OS provides. On the other hand, if you already have texinfo installed, try deactivating it; we've found many projects aren't yet compatible with texinfo 5.

Thank you ryan, the gnutls was successfully installed after I installed the texinfo port first.

comment:4 Changed 10 years ago by neverpanic (Clemens Lang)

That's weird – I'm on 10.9.4 as well, and /usr/bin/makeinfo --version tells me I have GNU texinfo 4.8, which succeeds in compiling and destrooting GnuTLS for me.

What do which -a makeinfo, /usr/bin/makeinfo --version and makeinfo --version print on your system?

comment:5 in reply to:  4 ; Changed 10 years ago by LeonardEighteen@…

Replying to cal@…:

That's weird – I'm on 10.9.4 as well, and /usr/bin/makeinfo --version tells me I have GNU texinfo 4.8, which succeeds in compiling and destrooting GnuTLS for me.

What do which -a makeinfo, /usr/bin/makeinfo --version and makeinfo --version print on your system?

Dear cal, thanks for your concern. The version of /usr/bin/makeinfo is 4.8 and the version of /opt/local/bin/makeinfo is 5.2. which -a makeinfo tells me the sequence is /opt/local/bin/makeinfo /usr/bin/makeinfo hence the makeinfo installed by MacPorts is used instead of the builtin one. So it is really weird that the builtin makeinfo 4.8 works fine for you but failed on my machine. Could you think about other reasons?

comment:6 in reply to:  5 Changed 10 years ago by LeonardEighteen@…

Replying to LeonardEighteen@…:

Replying to cal@…:

That's weird – I'm on 10.9.4 as well, and /usr/bin/makeinfo --version tells me I have GNU texinfo 4.8, which succeeds in compiling and destrooting GnuTLS for me.

What do which -a makeinfo, /usr/bin/makeinfo --version and makeinfo --version print on your system?

Dear cal, thanks for your concern. The version of /usr/bin/makeinfo is 4.8 and the version of /opt/local/bin/makeinfo is 5.2. which -a makeinfo tells me the sequence is /opt/local/bin/makeinfo /usr/bin/makeinfo hence the makeinfo installed by MacPorts is used instead of the builtin one. So it is really weird that the builtin makeinfo 4.8 works fine for you but failed on my machine. Could you think about other reasons? Should I remove what I have successfully installed as well as the texinfo 5.2 and try again the builtin one?

comment:7 Changed 10 years ago by neverpanic (Clemens Lang)

I guess I'll do a test build in a minute and compare my log with yours looking for differences. I recall there previously was some other documentation weirdness with GnuTLS related to timestamps, maybe the log diff will shed some light on that.

comment:8 Changed 10 years ago by Schamschula (Marius Schamschula)

This is strange. My systems all run 10.9.4 and have texinfo 5.2 installed. I just clean installed MacPorts on a 10.8.5 system. Without MacPorts texinfo 5.2, gnutls +dane @3.3.7 built w/o any problems.

Somehow the code related to documentation timestamps was deleted, along with an obsolete patch file, in the latest update. We might want to try reverting the pre-patch and post-patch code.

comment:9 Changed 10 years ago by neverpanic (Clemens Lang)

I'm thinking this might be a concurrency issue. Could the next person to experience this please try sudo port upgrade gnutls build.jobs=1 to see if that helps?

comment:10 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)

The error in the attached log occurs in the destroot phase. The build phase has already completed by that point, and the destroot phase is never run in parallel.

comment:11 in reply to:  8 Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to mschamschula@…:

Somehow the code related to documentation timestamps was deleted, along with an obsolete patch file, in the latest update. We might want to try reverting the pre-patch and post-patch code.

The two went hand in hand. In r122082, to fix #44323, I added a patch to fix problems with 5 headers. Updating the modification time of those files caused the build system to believe it needed to regenerate the corresponding documentation. Besides being an unnecessary waste of time, this failed to run properly, so I added the pre-patch block to save the files' mtimes and the post-patch block to restore them. When the patch was removed in r124479 and r124557, because it was integrated upstream, that made the pre-patch and post-patch code unnecessary which is why it was removed at the same time.

comment:12 Changed 4 years ago by chrstphrchvz (Christopher Chavez)

gnutls has since been update to 3.6.14. Is this issue still present, or can this ticket be closed?

comment:13 Changed 4 years ago by chrstphrchvz (Christopher Chavez)

Cc: chrstphrchvz added

comment:14 Changed 4 years ago by Schamschula (Marius Schamschula)

Resolution: worksforme
Status: newclosed

I don't think this is relevant to the current version of gnutls.

Note: See TracTickets for help on using tickets.